Fixed warnings from TestEnvironmentGetEnvironmentStrings
authorArmin Novak <armin.novak@thincast.com>
Tue, 29 Oct 2019 11:57:19 +0000 (12:57 +0100)
committerArmin Novak <armin.novak@thincast.com>
Tue, 29 Oct 2019 11:58:06 +0000 (12:58 +0100)
winpr/libwinpr/environment/test/TestEnvironmentGetEnvironmentStrings.c

index 238e5a2..b037a9d 100644 (file)
@@ -6,23 +6,33 @@
 
 int TestEnvironmentGetEnvironmentStrings(int argc, char* argv[])
 {
+       int r = -1;
        TCHAR* p;
-       int length;
+       size_t length;
        LPTCH lpszEnvironmentBlock;
 
+       WINPR_UNUSED(argc);
+       WINPR_UNUSED(argv);
+
        lpszEnvironmentBlock = GetEnvironmentStrings();
 
        p = (TCHAR*) lpszEnvironmentBlock;
 
        while (p[0] && p[1])
        {
-               _tprintf(_T("%s\n"), p);
+               const int rc = _tprintf(_T("%s\n"), p);
+               if (rc < 1)
+                       goto fail;
                length = _tcslen(p);
+               if (length != (size_t)(rc - 1))
+                       goto fail;
                p += (length + 1);
        }
 
+       r = 0;
+fail:
        FreeEnvironmentStrings(lpszEnvironmentBlock);
 
-       return 0;
+       return r;
 }