looks good On Sun, 2009-06-21 at 20:42 +0200, Jim Meyering wrote: > From: Jim Meyering <[email protected]> > > * services/msg.c (message_handler_req_exec_msg_queueopen): Remove > unnecessary "if" before free. > (message_handler_req_exec_msg_queueopenasync): Likewise. > (message_handler_req_exec_msg_messageget): Likewise. > --- > services/msg.c | 9 +++------ > 1 files changed, 3 insertions(+), 6 deletions(-) > > diff --git a/services/msg.c b/services/msg.c > index d4f7ec1..c0cf6dd 100644 > --- a/services/msg.c > +++ b/services/msg.c > @@ -2629,8 +2629,7 @@ error_exit: > list_add_tail (&cleanup->list, > &msg_pd->queue_cleanup_list); > } > else { > - if (cleanup) > - free (cleanup); > + free (cleanup); > } > > api->ipc_response_send ( > @@ -2799,8 +2798,7 @@ error_exit: > list_add_tail (&cleanup->list, > &msg_pd->queue_cleanup_list); > } > else { > - if (cleanup) > - free (cleanup); > + free (cleanup); > } > > api->ipc_response_send ( > @@ -3956,8 +3954,7 @@ error_exit: > } > } > > - if (msg != NULL) > - free (msg); > + free (msg); > } > > static void message_handler_req_exec_msg_messagedatafree (
_______________________________________________ Openais mailing list [email protected] https://lists.linux-foundation.org/mailman/listinfo/openais
