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

OpenMTP 3.2.0.6 on Monterey 12.6.1 talking to Pixel4XL, Kalam Mode: app pulls about 150 files and crashes, pane goes grey. POSSIBLE WORKAROUND #302

Closed
alecmuffett opened this issue Dec 10, 2022 · 5 comments
Labels
bug Something isn't working

Comments

@alecmuffett
Copy link

Spoiler

This seems only to affect Kalam-mode transfers; I re-ran the transfer under "Legacy" mode, and it worked.

Describe the bug

I fired up the app for the first time in several months; it autoupdated and restarted. Since the update, asking it to pull a DCIM folder from my phone to my desktop, it manages to pull about 150 files (variable per attempt) and then somehow crashes, leading to the OpenMTP pane blacking out although the menubar is still somewhat functional.

I repeated this 3x, same results each time.

I decided to try running it in a iTerm window, and the log is:

$ ./OpenMTP
objc[33898]: Class WebSwapCGLLayer is implemented in both /System/Library/Frameworks/WebKit.framework/Versions/A/Frameworks/WebCore.framework/Versions/A/Frameworks/libANGLE-shared.dylib (0x7ffb4a47fec8) and /Applications/OpenMTP.app/Contents/Frameworks/Electron Framework.framework/Versions/A/Libraries/libGLESv2.dylib (0x1137829c8). One of the two will be used. Which one is undefined.
2022/12/09 23:59:32 fatal error LIBUSB_ERROR_NOT_FOUND; closing connection.
Checking for update
Update for version 3.2.0 is not available (latest version: 3.2.0, downgrade is disallowed).
2022/12/09 23:59:47 OpenSession failed: LIBUSB_ERROR_TIMEOUT; attempting reset
2022/12/09 23:59:47 fatal error LIBUSB_ERROR_NOT_FOUND; closing connection.
2022/12/10 00:00:04 fatal error LIBUSB_ERROR_NOT_FOUND; closing connection.
libc++abi: terminating with uncaught exception of type Napi::Error: readPointer: Cannot read from nullptr pointer

The fatal "libusb" errors don't appear to actually be fatal since it pulls about 150 files, but the null pointer error comes up when it crashes properly.

Steps to Reproduce (for bugs)
Provide a link to a live example or an unambiguous set of steps to reproduce this bug. Include code to reproduce, if relevant

  1. Pull a large folder containing ~1100+ JPGs from an up-to-date Pixel phone, to a Monterey Mac, in Kalam mode.

Context
How has this issue affected you? What are you trying to accomplish?

File transfers of photos. Came to a complete stop.

@alecmuffett alecmuffett added the bug Something isn't working label Dec 10, 2022
@Saber-CC
Copy link

I have a similar situation to you.I have about 20,000 pictures to transfer, but every time I transfer thousands of pictures, the program will crash and turn gray.I tried issues#276 and nothing worked
I used your method to changed the mode to Legacy,and finally the transmission was successful.
This may a bug.

@ganeshrvel
Copy link
Owner

This seems like an issue with the node napi package that i am using. Unfortunately i am not getting enough time to find a fix for this issue.

@ganeshrvel
Copy link
Owner

ganeshrvel commented Dec 20, 2022

Thanks @alecmuffett for the report. Looks like there has been a regression in ffi-napi package unfortunately I cannot revert back the package upgrade. I have reported the issue with them: node-ffi-napi/node-ffi-napi#235.

Meanwhile I am looking into migrating the ffi package to something else which is stable.

@ganeshrvel
Copy link
Owner

ganeshrvel commented Jan 25, 2023

OpenMTP v3.2.10-beta.2 is now rolling out.

Note: This a pre-release version and it might be unstable. ⚠️

Features

  • Fixes - the app crash while downloading files
  • Fixes - the app crash on older versions of macOS
  • Fixes - the app crash caused while transferring files a large number of files.

If you are on an older version of OpenMTP and have frozen the auto-app update option for the sake of preventing the app crash, you may re-enable it from Settings > Updates > Automatically Check for Updates

Note: We have now officially retired the support for Kalam Kernel on macOS 10.13 (OS X El High Sierra) and lower. Though the "Legacy" MTP mode will continue working on these outdated machines.
Read more: https://github.com/ganeshrvel/openmtp/tree/release/3.2.10-beta.1#system-requirements-and-support

Download:

@ganeshrvel
Copy link
Owner

Fixed the issue in #314. Closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants