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

crash when URL-encoding empty parentheses () in UTF-8 #25

Closed
jeichberger opened this issue Oct 17, 2023 · 1 comment
Closed

crash when URL-encoding empty parentheses () in UTF-8 #25

jeichberger opened this issue Oct 17, 2023 · 1 comment
Assignees

Comments

@jeichberger
Copy link

NPP crashes when trying to URL-encode two parentheses characters in utf-8.

Steps to reproduce:

  • create or open a file
  • set Encoding to UTF-8
  • type empty parentheses () anywhere in the file
  • select exactly these parentheses
  • click on Plugins -> MIME Tools -> Full URL Encode
  • NPP crashes

Environment:

  • Windows 11 Enterprise, 22H2
  • Notepad++ v8.5.7 (64-bit)
  • Mime tools 2.9
@gerdb42
Copy link

gerdb42 commented Dec 1, 2023

Same issue here, though I can't reliably reproduce the crash. Occasionally, NPP crashes after using "URL Encode". Also the crash does not always happen immediately, but on a subsequent action. The following is shown in the Windows Application Log:

Faulting application name: notepad++.exe, version: 8.6.0.0, time stamp: 0x655f7686
Faulting module name: ntdll.dll, version: 10.0.19041.3693, time stamp: 0x3ab9e7d5
Exception code: 0xc0000374
Fault offset: 0x000e6d13
Faulting process id: 0x22a0
Faulting application start time: 0x01da2469136523af
Faulting application path: C:\Program Files (x86)\Tools\Notepad++\notepad++.exe
Faulting module path: C:\WINDOWS\SYSTEM32\ntdll.dll
Report Id: d95a2529-6b95-4cd5-9b87-717b039715de
Faulting package full name: 
Faulting package-relative application ID: 

@donho donho self-assigned this Dec 17, 2023
donho added a commit to donho/mimetools that referenced this issue Dec 17, 2023
@donho donho closed this as completed in 2c7c94d Dec 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants