From f965db5fc1799c2ae6adfe92896eb315a0c3447c Mon Sep 17 00:00:00 2001 From: Dawid Gajownik Date: Sat, 1 Aug 2015 11:02:54 -0300 Subject: [PATCH] compositor-rdp: free() can handle NULL pointers There's no need to check if a pointer exists before passing it to free(). free() can handle NULL pointers. Signed-off-by: Dawid Gajownik Reviewed-By: David FORT --- src/compositor-rdp.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/compositor-rdp.c b/src/compositor-rdp.c index 4ef0dd15..6eac7f90 100644 --- a/src/compositor-rdp.c +++ b/src/compositor-rdp.c @@ -1233,12 +1233,9 @@ err_output: err_compositor: weston_compositor_shutdown(compositor); err_free_strings: - if (b->rdp_key) - free(b->rdp_key); - if (b->server_cert) - free(b->server_cert); - if (b->server_key) - free(b->server_key); + free(b->rdp_key); + free(b->server_cert); + free(b->server_key); free(b); return NULL; }