M2Team / NanaZip

The 7-Zip derivative intended for the modern Windows experience
https://sourceforge.net/projects/nanazip/
Other
7.75k stars 204 forks source link

App crash when you scroll or the type of file organization #408

Open quietrobot opened 1 month ago

quietrobot commented 1 month ago

I don't know if it is related to the number of folders, but automatically when using the scroll (mouse or bar) the app crashes. Windows 11 22631.3593

quietrobot commented 1 month ago

update: It also happens when changing the view options: large icons, details, etc.

dinhngtu commented 1 month ago

Could you post a crash dump?

Open Registry Editor, create a key with the following path: HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\Windows Error Reporting\LocalDumps\NanaZip.Windows.exe

In this key, create a DWORD value named DumpType with a value of 2.

Try to reproduce the crash, a crash dump will be produced in %localappdata%\CrashDumps.

Also look for app crash events in Event Viewer.

quietrobot commented 1 month ago

@dinhngtu that generated several .dmp files, what do I do with them? That is the event viewer info (sorry for the Spanish).

image

dinhngtu commented 1 month ago

Could you compress and upload them here?

quietrobot commented 1 month ago

sure

NanaZip.Modern.exe.24456.dmp.zip

dinhngtu commented 1 month ago

Could you uninstall TranslucentFlyouts and try again?

liwenjie119 commented 1 month ago

CrashDumps.zip 点提取或选项-编辑器时同样闪退,无论正式版或者preview。 CrashDumps_2.zip

quietrobot commented 1 month ago

Could you uninstall TranslucentFlyouts and try again?

Correct, without TranslucentFlyouts the app does not crash.

dinhngtu commented 1 month ago

Could you uninstall TranslucentFlyouts and try again?

Correct, without TranslucentFlyouts the app does not crash.

Which version of TranslucentFlyouts were you using? I'll add it to the DLL blocker.

CrashDumps.zip 点提取或选项-编辑器时同样闪退,无论正式版或者preview。 CrashDumps_2.zip

This is due to a bug of the DLL blocker when dealing with Sogou Pinyin input. I'll try to fix it.

MouriNaruto commented 1 month ago

I have merged the fix PR. But I won't close this issue until the new version is released.

Kenji Mouri

yyjdelete commented 1 month ago

Add a crash dump for iFlyIME(讯飞输入法)

For some unknown reason, it does't tigger Windows Error Reporting, try to make a dump with visual studio

Download all parts of the 7z, and remove .zip from filename to extract it

NanaZip.Modern.7z.001.zip NanaZip.Modern.7z.002.zip NanaZip.Modern.7z.003.zip NanaZip.Modern.7z.004.zip NanaZip.Modern.7z.005.zip

MouriNaruto commented 1 month ago

@yyjdelete Currently, could you help us test the latest main branch implementation because the fix PR has been merged.

Kenji Mouri

dinhngtu commented 1 month ago

The iFlyIME crash is a separate issue:

ERROR_CODE: (NTSTATUS) 0xc0000017 - {Not Enough Quota}  Not enough virtual memory or paging file quota is available to complete the specified operation.

    KernelBase.dll!RaiseException()
    BaseGUI.dll!000000005e01afbc()  void * __cdecl ATL::__AllocStdCallThunk(void)
    BaseGUI.dll!000000005df49a73()
    BaseGUI.dll!000000005dfbe3a8()
    BaseGUI.dll!000000005dfbe368()
    BaseGUI.dll!000000005dfbed5b()
    BaseGUI.dll!000000005dfbea77()
    BaseGUI.dll!000000005dfbe783()
    BaseGUI.dll!000000005def8021()
    BaseGUI.dll!000000005deed9de()
    iFlyTsfUI.dll!000000005c9e3e90()
    iFlyServer.dll!000000005d686a8d()
    iFlyServer.dll!000000005d684809()
    iFlyServer.dll!000000005d685956()
    iFlyServer.dll!000000005d683a1d()
    iFlyServer.dll!000000005d665d13()
    iFlyServer.dll!000000005d644eb0()
    iFlyServer.dll!000000005d646cb2()
    iFlyServer.dll!000000005d645f42()
    iFlyServer.dll!000000005d643ad8()
    iFlyServer.dll!000000005d65c68c()
    iFlyBaseLogic.dll!000000005e28f3ed()
    iFlyBaseLogic.dll!000000005e28c019()
    iFlyBaseLogic.dll!000000005e28dc47()
    iFlyBaseLogic.dll!000000005e28d925()
    iFlyBaseLogic.dll!000000005e28dbe7()
    iflycommon.dll!000000005db8f847()
    iflycommon.dll!000000005db8d68c()
    iflycommon.dll!000000005db8df08()
    iflycommon.dll!000000005dba37a1()
    iflycommon.dll!000000005dba4a08()
    iflycommon.dll!000000005dba4970()
    iflycommon.dll!000000005db8f344()
    iflycommon.dll!000000005db8f1df()
    iflycommon.dll!000000005dbb8579()
    iflycommon.dll!000000005db8f19d()
    iflycommon.dll!000000005db87ff8()
    iflycommon.dll!000000005db887a2()
    iflycommon.dll!000000005db9d5ac()
    kernel32.dll!BaseThreadInitThunk()
    ntdll.dll!RtlUserThreadStart()

BaseGUI.dll ATL::__AllocStdCallThunk allocates an ATL thunk mapping with PAGE_EXECUTE_READWRITE, which is not allowed by Arbitrary Code Guard. This causes it to call RaiseException.

@MouriNaruto I think there are 2 options:

quietrobot commented 1 month ago

Could you uninstall TranslucentFlyouts and try again?

Correct, without TranslucentFlyouts the app does not crash.

Which version of TranslucentFlyouts were you using? I'll add it to the DLL blocker.

Thanks, translucentFlyoutsV2

yyjdelete commented 1 month ago

@yyjdelete Currently, could you help us test the latest main branch implementation because the fix PR has been merged.

Kenji Mouri

@MouriNaruto It no longer crash with Sogou Pinyin, and still crash with iFlyIME Tested with Release\NanaZipPackage\x64\NanaZip.Modern.exe in the CI build

MouriNaruto commented 1 month ago

@yyjdelete Got it.

@dinhngtu I think similar issues will become more and more. Maybe we need to consider about more generic. For example, except for the NanaZip application folder, modules that don't have signatures from Microsoft, Microsoft Store, or WHQL will be banned.

Kenji Mouri

dinhngtu commented 1 month ago

@dinhngtu I think similar issues will become more and more. Maybe we need to consider about more generic. For example, except for the NanaZip application folder, modules that don't have signatures from Microsoft, Microsoft Store, or WHQL will be banned.

@MouriNaruto I think we should at least allow input methods and accessibility tools to serve users' needs.

How about this procedure:

MouriNaruto commented 1 month ago

@dinhngtu

You are right about that, and I have merged your related PR.

Kenji Mouri