upstream/opnsense-src
1
0
Fork 0
mirror of https://github.com/opnsense/src.git synced 2026-06-09 00:32:25 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

mixer: Fix a bogus free() call in the main loop

Browse source 
After a strsep() call, p might not point at the original allocation
anymore.

Reported by:	CHERI
Reviewed by:	christos
MFC after:	1 week
Differential Revision:	https://reviews.freebsd.org/D47266

(cherry picked from commit c22be0b181)
This commit is contained in:
Mark Johnston 2024-10-23 16:55:39 +00:00
parent 90d8c39d22
commit bf2145f361
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
usr.sbin/mixer/mixer.c
View file

@ -147,7 +147,9 @@ main(int argc, char *argv[])
parse:
while (argc > 0) {
if ((p = strdup(*argv)) == NULL)
char *orig;
if ((orig = p = strdup(*argv)) == NULL)
err(1, "strdup(%s)", *argv);
/* Check if we're using the shorthand syntax for volume setting. */
@ -200,7 +202,7 @@ parse:
/* Input: `dev.control=val`. */
cp->mod(cp->parent_dev, valstr);
next:
free(p);
free(orig);
argc--;
argv++;
}