From faf0947ebbcdbd2088d6349e936a23b24d09153b Mon Sep 17 00:00:00 2001 From: "Daniel P. Berrange" Date: Mon, 12 Dec 2011 17:53:04 +0000 Subject: Remove useless if() before free() The free() function allows NULL to be passed in, so any code which puts a if() before free() is wasting time --- server/mjpeg_encoder.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'server/mjpeg_encoder.c') diff --git a/server/mjpeg_encoder.c b/server/mjpeg_encoder.c index 599d4f57..6b68549c 100644 --- a/server/mjpeg_encoder.c +++ b/server/mjpeg_encoder.c @@ -129,8 +129,7 @@ static boolean empty_mem_output_buffer(j_compress_ptr cinfo) memcpy(nextbuffer, dest->buffer, dest->bufsize); - if (dest->newbuffer != NULL) - free(dest->newbuffer); + free(dest->newbuffer); dest->newbuffer = nextbuffer; -- cgit