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 <gajownik@gmail.com>
Reviewed-By: David FORT <contact@hardening-consulting.com>

1 files 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;
 }