msvcrt: Remove redundant NULL checks before free().

Michael Stefaniuc mstefani at redhat.de
Fri Jul 8 03:09:32 CDT 2016


Signed-off-by: Michael Stefaniuc <mstefani at redhat.de>
---
 dlls/msvcrt/errno.c | 2 +-
 dlls/msvcrt/file.c  | 3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/dlls/msvcrt/errno.c b/dlls/msvcrt/errno.c
index 46b1859..893ed7b 100644
--- a/dlls/msvcrt/errno.c
+++ b/dlls/msvcrt/errno.c
@@ -365,7 +365,7 @@ void CDECL MSVCRT__wperror(const MSVCRT_wchar_t* str)
         }
     }
     MSVCRT_perror(buffer);
-    if (buffer) MSVCRT_free(buffer);
+    MSVCRT_free(buffer);
 }
 
 /*********************************************************************
diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c
index 5a2f1e6..432d5c7 100644
--- a/dlls/msvcrt/file.c
+++ b/dlls/msvcrt/file.c
@@ -3498,8 +3498,7 @@ int CDECL MSVCRT__write(int fd, const void* buf, unsigned int count)
         if (!WriteFile(hand, q, size, &num_written, NULL))
             num_written = -1;
         release_ioinfo(info);
-        if(p)
-            MSVCRT_free(p);
+        MSVCRT_free(p);
         if (num_written != size)
         {
             TRACE("WriteFile (fd %d, hand %p) failed-last error (%d), num_written %d\n",
-- 
2.5.5



More information about the wine-patches mailing list