[PATCH 0/1] MR7874: winedump: Fix a crash in 'dump' command.

[Nikolay Sivov (＠nsivov)]

This is a regression from 405eb2a386df0135174c5d04a24cb3b1adc1c710.
Now with two kinds of handlers we need to check for nulls in both loops.

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com>

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7874

[PATCH 1/1] winedump: Fix a crash in 'dump' command.

[Nikolay Sivov]

From: Nikolay Sivov <nsivov(a)codeweavers.com>

This is a regression from 405eb2a386df0135174c5d04a24cb3b1adc1c710.
Now with two kinds of handlers we need to check for nulls in both loops.

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com>
---
 tools/winedump/dump.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tools/winedump/dump.c b/tools/winedump/dump.c
index 6ff4c0b58a4..102d1427413 100644
--- a/tools/winedump/dump.c
+++ b/tools/winedump/dump.c
@@ -285,6 +285,7 @@ BOOL dump_analysis(const char *name, file_dumper fn, enum FileSig wanted_sig)
 
         for (dpr = dumpers; dpr->kind != SIG_UNKNOWN; dpr++)
         {
+            if (!dpr->get_kind) continue;
             if (dpr->get_kind() == dpr->kind &&
                 (wanted_sig == SIG_UNKNOWN || wanted_sig == dpr->kind))
             {
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/7874