Fixed array out of bound access.
authorArmin Novak <armin.novak@thincast.com>
Tue, 1 Sep 2015 09:44:15 +0000 (11:44 +0200)
committerArmin Novak <armin.novak@thincast.com>
Tue, 1 Sep 2015 10:25:43 +0000 (12:25 +0200)
winpr/libwinpr/environment/test/TestEnvironmentGetSetEB.c

index 1134140..6da2034 100644 (file)
@@ -99,11 +99,11 @@ int TestEnvironmentGetSetEB(int argc, char* argv[])
        }
        free(lpszEnvironmentBlockNew);
 
-       lpszEnvironmentBlockNew = (LPTCH) malloc(1024);
+       lpszEnvironmentBlockNew = (LPTCH) malloc(length);
        if (!lpszEnvironmentBlockNew)
                return -1;
 
-       memcpy(lpszEnvironmentBlockNew,lpszEnvironmentBlock,56);
+       memcpy(lpszEnvironmentBlockNew,lpszEnvironmentBlock,length);
 
        /* Set variable in empty environment block */
        if (SetEnvironmentVariableEBA(&lpszEnvironmentBlockNew, "test", "5"))