Jason Edmeades : cmd: Fix issue in WCMD_assoc highlighted by valgrind.

Alexandre Julliard julliard at winehq.org
Thu Sep 13 03:49:15 CDT 2018


Module: wine
Branch: stable
Commit: 554f6cf9aa01af54fb6d5aeed48394ae5de8bed5
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=554f6cf9aa01af54fb6d5aeed48394ae5de8bed5

Author: Jason Edmeades <us at edmeades.me.uk>
Date:   Wed Jul  4 22:08:40 2018 +0100

cmd: Fix issue in WCMD_assoc highlighted by valgrind.

In an error condition, the wrong variable was being used for an insert,
resulting in a read from uninitialized data. This could be triggered for
example by 'ftype jason=', and the error message should have included
jason but instead was just ''.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=38849
Signed-off-by: Jason Edmeades <us at edmeades.me.uk>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
(cherry picked from commit ab46a89c3b8649844a70ee30abffbbd275128bc3)
Signed-off-by: Michael Stefaniuc <mstefani at winehq.org>

---

 programs/cmd/builtins.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/programs/cmd/builtins.c b/programs/cmd/builtins.c
index 04b098e..3648f29 100644
--- a/programs/cmd/builtins.c
+++ b/programs/cmd/builtins.c
@@ -4883,7 +4883,7 @@ void WCMD_assoc (const WCHAR *args, BOOL assoc) {
               LoadStringW(hinst, WCMD_NOFTYPE, msgbuffer,
                           sizeof(msgbuffer)/sizeof(WCHAR));
             }
-            WCMD_output_stderr(msgbuffer, keyValue);
+            WCMD_output_stderr(msgbuffer, args);
             errorlevel = 2;
           }
 




More information about the wine-cvs mailing list