aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <mchehab+huawei@kernel.org>2021-02-23 09:23:44 +0100
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>2021-02-23 09:23:44 +0100
commita6e69e4dcfd4a3b6543e89ac7c39089a7db42c75 (patch)
treece5b5c870a727d3c6bacc1a0dc79a29470ed9767
parent50f3254c2356927e2232f63bcf1b3b49c70cded2 (diff)
libdvbv5: dvb-dev-remote: don't leak resources at send_buf()
If errors occur, ensure that the message buffer will be freed. Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
-rw-r--r--lib/libdvbv5/dvb-dev-remote.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/lib/libdvbv5/dvb-dev-remote.c b/lib/libdvbv5/dvb-dev-remote.c
index 07ab35a6..8935db4d 100644
--- a/lib/libdvbv5/dvb-dev-remote.c
+++ b/lib/libdvbv5/dvb-dev-remote.c
@@ -347,6 +347,7 @@ static struct queued_msg *send_buf(struct dvb_device_priv *dvb, int fd,
dvb_logdbg("buffer to short for int32_t");
stack_dump(parms);
pthread_mutex_unlock(&priv->lock_io);
+ free(msg);
return NULL;
}
memcpy(p, &i32, 4);
@@ -359,6 +360,7 @@ static struct queued_msg *send_buf(struct dvb_device_priv *dvb, int fd,
p - buf, len, sizeof(buf));
stack_dump(parms);
pthread_mutex_unlock(&priv->lock_io);
+ free(msg);
return NULL;
}
i32 = htobe32(len);
@@ -372,6 +374,7 @@ static struct queued_msg *send_buf(struct dvb_device_priv *dvb, int fd,
dvb_logdbg("buffer to big!");
stack_dump(parms);
pthread_mutex_unlock(&priv->lock_io);
+ free(msg);
return NULL;
}

Privacy Policy