linux-kernel - Re: [PATCH v3 12/12] ALSA: hda/cs35l56: Add driver for Cirrus Logic CS35L56 amplifier

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4ec68c42-dbab-5006-c6a5-0be147a7d0f1@microchip.com>
Date:   Tue, 30 May 2023 07:39:04 +0000
From:   <Claudiu.Beznea@...rochip.com>
To:     <rf@...nsource.cirrus.com>, <tiwai@...e.com>, <broonie@...nel.org>,
        <perex@...ex.cz>
CC:     <alsa-devel@...a-project.org>, <linux-kernel@...r.kernel.org>,
        <patches@...nsource.cirrus.com>, <simont@...nsource.cirrus.com>
Subject: Re: [PATCH v3 12/12] ALSA: hda/cs35l56: Add driver for Cirrus Logic
 CS35L56 amplifier

On 26.05.2023 15:28, Richard Fitzgerald wrote:
> +static int cs35l56_hda_request_firmware_file(struct cs35l56_hda *cs35l56,
> +                                            const struct firmware **firmware, char **filename,
> +                                            const char *dir, const char *system_name,
> +                                            const char *amp_name,
> +                                            const char *filetype)
> +{
> +       char *s, c;
> +       int ret = 0;
> +
> +       if (system_name && amp_name)
> +               *filename = kasprintf(GFP_KERNEL, "%scs35l56%s-%02x-dsp1-misc-%s-%s.%s", dir,
> +                                     cs35l56->base.secured ? "s" : "", cs35l56->base.rev,
> +                                     system_name, amp_name, filetype);
> +       else if (system_name)
> +               *filename = kasprintf(GFP_KERNEL, "%scs35l56%s-%02x-dsp1-misc-%s.%s", dir,
> +                                     cs35l56->base.secured ? "s" : "", cs35l56->base.rev,
> +                                     system_name, filetype);
> +       else
> +               *filename = kasprintf(GFP_KERNEL, "%scs35l56%s-%02x-dsp1-misc.%s", dir,
> +                                     cs35l56->base.secured ? "s" : "", cs35l56->base.rev,
> +                                     filetype);
> +
> +       if (!*filename)
> +               return -ENOMEM;
> +
> +       /*
> +        * Make sure that filename is lower-case and any non alpha-numeric
> +        * characters except full stop and forward slash are replaced with
> +        * hyphens.
> +        */
> +       s = *filename;
> +       while (*s) {
> +               c = *s;
> +               if (isalnum(c))
> +                       *s = tolower(c);
> +               else if (c != '.' && c != '/')
> +                       *s = '-';
> +               s++;
> +       }
> +
> +       ret = firmware_request_nowarn(firmware, *filename, cs35l56->base.dev);
> +       if (ret) {
> +               dev_dbg(cs35l56->base.dev, "Failed to request '%s'\n", *filename);
> +               kfree(*filename);
> +               *filename = NULL;
> +       } else {
> +               dev_dbg(cs35l56->base.dev, "Found '%s'\n", *filename);

I may be wrong but *filename seems leaked to me on this path. I noticed it
is could be duplicated on cs_dsp_debugfs_save_wmfwname() but the original
copy seems left aside.

> +       }
> +
> +       return ret;
> +}