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

Alexandre Julliard julliard at winehq.org
Thu Jul 5 15:25:53 CDT 2018


Module: wine
Branch: master
Commit: ab46a89c3b8649844a70ee30abffbbd275128bc3
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=ab46a89c3b8649844a70ee30abffbbd275128bc3

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>

---

 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 3a0981d..57a41c4 100644
--- a/programs/cmd/builtins.c
+++ b/programs/cmd/builtins.c
@@ -4923,7 +4923,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