Le 01/08/2015 16:02, Dawid Gajownik a écrit : > 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 <[email protected]>
Reviewed-By: David FORT <[email protected]> > --- > 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 a83948b..b5169a8 100644 > --- a/src/compositor-rdp.c > +++ b/src/compositor-rdp.c > @@ -1244,12 +1244,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; > } > -- David FORT website: http://www.hardening-consulting.com/ _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
