From 9839813e7c283ccc0d2a09279df0123d575d82d3 Mon Sep 17 00:00:00 2001 From: Takashi Iwai Date: Fri, 9 Dec 2016 17:28:47 +0100 Subject: [PATCH] alsactl: Fix potential NULL dereferences in daemon mode The code releasing the each card object may access to NULL when a bogus count is given. Add a NULL check just to make sure. Signed-off-by: Takashi Iwai --- alsactl/daemon.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/alsactl/daemon.c b/alsactl/daemon.c index 127701e..4f0a935 100644 --- a/alsactl/daemon.c +++ b/alsactl/daemon.c @@ -451,8 +451,10 @@ save: out: free(pfd); remove(pidfile); - for (i = 0; i < count; i++) - card_free(&cards[i]); - free(cards); + if (cards) { + for (i = 0; i < count; i++) + card_free(&cards[i]); + free(cards); + } return 0; }