ASoC: core: clarify the driver name initialization
authorJaroslav Kysela <perex@perex.cz>
Thu, 29 Sep 2022 14:37:54 +0000 (16:37 +0200)
committerMark Brown <broonie@kernel.org>
Thu, 29 Sep 2022 16:22:41 +0000 (17:22 +0100)
The driver field in the struct snd_ctl_card_info is a valid
user space identifier. Actually, many ASoC drivers do not care
and let to initialize this field using a standard wrapping method.
Unfortunately, in this way, this field becomes unusable and
unreadable for the drivers with longer card names. Also,
there is a possibility to have clashes (driver field has
only limit of 15 characters).

This change will print an error when the wrapping is used.
The developers of the affected drivers should fix the problem.

Signed-off-by: Jaroslav Kysela <perex@perex.cz>
Reviewed-by: NĂ­colas F. R. A. Prado <nfraprado@collabora.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/soc-core.c

index a812487..12a82f5 100644 (file)
@@ -1836,21 +1836,22 @@ match:
        }
 }
 
-#define soc_setup_card_name(name, name1, name2, norm)          \
-       __soc_setup_card_name(name, sizeof(name), name1, name2, norm)
-static void __soc_setup_card_name(char *name, int len,
-                                 const char *name1, const char *name2,
-                                 int normalization)
+#define soc_setup_card_name(card, name, name1, name2) \
+       __soc_setup_card_name(card, name, sizeof(name), name1, name2)
+static void __soc_setup_card_name(struct snd_soc_card *card,
+                                 char *name, int len,
+                                 const char *name1, const char *name2)
 {
+       const char *src = name1 ? name1 : name2;
        int i;
 
-       snprintf(name, len, "%s", name1 ? name1 : name2);
+       snprintf(name, len, "%s", src);
 
-       if (!normalization)
+       if (name != card->snd_card->driver)
                return;
 
        /*
-        * Name normalization
+        * Name normalization (driver field)
         *
         * The driver name is somewhat special, as it's used as a key for
         * searches in the user-space.
@@ -1870,6 +1871,14 @@ static void __soc_setup_card_name(char *name, int len,
                        break;
                }
        }
+
+       /*
+        * The driver field should contain a valid string from the user view.
+        * The wrapping usually does not work so well here. Set a smaller string
+        * in the specific ASoC driver.
+        */
+       if (strlen(src) > len - 1)
+               dev_err(card->dev, "ASoC: driver name too long '%s' -> '%s'\n", src, name);
 }
 
 static void soc_cleanup_card_resources(struct snd_soc_card *card)
@@ -2037,12 +2046,12 @@ static int snd_soc_bind_card(struct snd_soc_card *card)
        /* try to set some sane longname if DMI is available */
        snd_soc_set_dmi_name(card, NULL);
 
-       soc_setup_card_name(card->snd_card->shortname,
-                           card->name, NULL, 0);
-       soc_setup_card_name(card->snd_card->longname,
-                           card->long_name, card->name, 0);
-       soc_setup_card_name(card->snd_card->driver,
-                           card->driver_name, card->name, 1);
+       soc_setup_card_name(card, card->snd_card->shortname,
+                           card->name, NULL);
+       soc_setup_card_name(card, card->snd_card->longname,
+                           card->long_name, card->name);
+       soc_setup_card_name(card, card->snd_card->driver,
+                           card->driver_name, card->name);
 
        if (card->components) {
                /* the current implementation of snd_component_add() accepts */