git: 64f4e2db6d19 - main - sound: Correctly check nvlist_unpack() error
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Mon, 20 May 2024 14:18:48 UTC
The branch main has been updated by christos:
URL: https://cgit.FreeBSD.org/src/commit/?id=64f4e2db6d19d8ab520903a197fcaa8cc7ab9f9a
commit 64f4e2db6d19d8ab520903a197fcaa8cc7ab9f9a
Author: Christos Margiolis <christos@FreeBSD.org>
AuthorDate: 2024-05-20 14:18:33 +0000
Commit: Christos Margiolis <christos@FreeBSD.org>
CommitDate: 2024-05-20 14:18:33 +0000
sound: Correctly check nvlist_unpack() error
The current check is never false and if nvlist_unpack() fails, we might
panic later down the road.
PR: 266144
Sponsored by: The FreeBSD Foundation
MFC after: 1 day
Reviewed by: dev_submerge.ch, emaste
Differential Revision: https://reviews.freebsd.org/D45237
---
sys/dev/sound/pcm/sndstat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sys/dev/sound/pcm/sndstat.c b/sys/dev/sound/pcm/sndstat.c
index f310d8f3bff3..bbcb2fda7c29 100644
--- a/sys/dev/sound/pcm/sndstat.c
+++ b/sys/dev/sound/pcm/sndstat.c
@@ -705,7 +705,7 @@ sndstat_unpack_user_nvlbuf(const void *unvlbuf, size_t nbytes, nvlist_t **nvl)
}
*nvl = nvlist_unpack(nvlbuf, nbytes, 0);
free(nvlbuf, M_DEVBUF);
- if (nvl == NULL) {
+ if (*nvl == NULL) {
return (EINVAL);
}