Lionel Debroux : server: Remove redundant NULL checks before free ( found by Smatch).

Alexandre Julliard julliard at winehq.org
Mon Oct 22 09:55:19 CDT 2007


Module: wine
Branch: master
Commit: 2f89fe7b37ebe63b285aa88f87f54b7995a9fa77
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=2f89fe7b37ebe63b285aa88f87f54b7995a9fa77

Author: Lionel Debroux <lionel_debroux at yahoo.fr>
Date:   Sat Oct 20 09:35:13 2007 +0200

server: Remove redundant NULL checks before free (found by Smatch).

---

 server/console.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/server/console.c b/server/console.c
index a94c881..61c4924 100644
--- a/server/console.c
+++ b/server/console.c
@@ -710,7 +710,7 @@ static int set_console_input_info( const struct set_console_input_info_request *
 	console->history_index -= delta;
 
 	for (i = 0; i < console->history_size; i++)
-	    if (console->history[i]) free( console->history[i] );
+	    free( console->history[i] );
 	free( console->history );
 	console->history = mem;
 	console->history_size = req->history_size;
@@ -1033,7 +1033,7 @@ static void console_input_destroy( struct object *obj )
     release_object( console_in->event );
 
     for (i = 0; i < console_in->history_size; i++)
-        if (console_in->history[i]) free( console_in->history[i] );
+        free( console_in->history[i] );
     free( console_in->history );
 }
 




More information about the wine-cvs mailing list