msvcrt: Remove redundant NULL checks before free().

Signed-off-by: Michael Stefaniuc <mstefani@redhat.de> Signed-off-by: Piotr Caban <piotr@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

msvcrt: Remove redundant NULL checks before free().
dbafbccc · Michael Stefaniuc · Alexandre Julliard · fb1f4275 · dbafbccc · dbafbccc
Commit dbafbccc authored Jul 08, 2016 by Michael Stefaniuc Committed by Alexandre Julliard Jul 08, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 3 deletions

errno.c dlls/msvcrt/errno.c +1 -1

file.c dlls/msvcrt/file.c +1 -2

No files found.
--- 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);
 }

 /*********************************************************************

--- 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",