Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NULL_POINTER_READ_c0000005_SystemInformer.exe!PhAddIListViewColumn #2364

Closed
MagicAndre1981 opened this issue Jan 4, 2025 · 2 comments · Fixed by #2365
Closed

NULL_POINTER_READ_c0000005_SystemInformer.exe!PhAddIListViewColumn #2364

MagicAndre1981 opened this issue Jan 4, 2025 · 2 comments · Fixed by #2365
Labels

Comments

@MagicAndre1981
Copy link
Contributor

Brief description of your issue

I tried to open GraphicsDeviceDetails on Win 11 24H2 and Si crashed:

 # Child-SP          RetAddr               Call Site
00 000000b5`d10ff0d0 00007ffd`7af0be50     SystemInformer!PhAddIListViewColumn(struct IListView * ListView = 0x00000000`00000000, long Index = 0n0, long DisplayIndex = 0n0, long SubItemIndex = 0n0, long Format = 0n0, long Width = 0n230, wchar_t * Text = 0x00007ffd`7af409c0 "Property")+0x28 [phlib\guisuplistview.cpp @ 112] 
01 000000b5`d10ff180 00007ffd`afe10f00     HardwareDevices!GraphicsDeviceDetailsDlgProc(struct HWND__ * hwndDlg = 0x00000000`000904e8, unsigned int uMsg = <Value unavailable error>, unsigned int64 wParam = 0xa0472, int64 lParam = 0n2508154927936)+0x370 [plugins\HardwareDevices\gpudetails.c @ 382] 
02 000000b5`d10ff210 00007ffd`afe0f454     user32!UserCallDlgProcCheckWow+0x1e4
03 000000b5`d10ff300 00007ffd`afe0f376     user32!DefDlgProcWorker+0xc4
04 000000b5`d10ff3b0 00007ffd`afe15801     user32!DefDlgProcW+0x36
05 000000b5`d10ff3f0 00007ffd`afe14b23     user32!UserCallWinProcCheckWow+0x341
06 000000b5`d10ff550 00007ffd`afe1eb29     user32!SendMessageWorker+0x223
07 000000b5`d10ff600 00007ffd`afe1df27     user32!InternalCreateDialog+0xa69
08 000000b5`d10ff7e0 00007ffd`afe1f4a8     user32!CreateDialogIndirectParamAorW+0x57
09 000000b5`d10ff830 00007ff6`dc4392b2     user32!CreateDialogIndirectParamW+0x18
0a 000000b5`d10ff870 00007ffd`7af0bffb     SystemInformer!PhCreateDialog(void * Instance = 0x00007ffd`7af00000, wchar_t * Template = <Value unavailable error>, struct HWND__ * ParentWindow = 0x00000000`000d0358, <function> * DialogProc = 0x00007ffd`7af0bae0, void * Parameter = 0x00000247`f9aefb40)+0x92 [phlib\guisup.c @ 1561] 
0b 000000b5`d10ff910 00007ff6`dc42a735     HardwareDevices!GraphicsDeviceDetailsDialogThreadStart(void * Parameter = 0x00000247`f9aefb40)+0x7b [plugins\HardwareDevices\gpudetails.c @ 543] 
0c 000000b5`d10ffbc0 00007ffd`af72dbe7     SystemInformer!PhpBaseThreadStart(void * Parameter = <Value unavailable error>)+0x75 [phlib\basesup.c @ 195] 
0d 000000b5`d10ffc00 00007ffd`b07ffbec     kernel32!BaseThreadInitThunk+0x17
0e 000000b5`d10ffc30 00000000`00000000     ntdll!RtlUserThreadStart+0x2c

image

Here is the calling code before crash:

image

Steps to reproduce (optional)

No response

Expected behavior (optional)

No response

Actual behavior (optional)

No response

Environment (optional)

3.2.25004.614
Windows 11 24H2 x64
@ge0rdi
Copy link
Contributor

ge0rdi commented Jan 4, 2025

Looks like the bug was introduced in f8757ab.

That commit added context->ListViewClass = PhGetListViewInterface(hwndDlg); line.
But there should be context->ListViewHandle passed instead of hwndDlg.

ge0rdi added a commit to ge0rdi/systeminformer that referenced this issue Jan 4, 2025
Regression introduced in f8757a.

Fixes winsiderss#2364.
@jxy-s jxy-s closed this as completed in c79ad2a Jan 5, 2025
@MagicAndre1981
Copy link
Contributor Author

works fine with 3.2.25011.2103 (bfbd659)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants