View Issue Details

IDProjectCategoryView StatusLast Update
0017703MMW 5Syncpublic2021-04-07 21:38
Reporterrusty Assigned To 
PriorityurgentSeveritymajorReproducibilitysometimes
Status closedResolutionfixed 
Product Version5.0 
Target Version5.0Fixed in Version5.0 
Summary0017703: Invalid pointer on sync
DescriptionOccasionally, when syncing certain playlists to a USB key, MM has gotten stuck.

Upon further investigation, it seems that it often temporarily gets stuck on Air - Dead Bodies, though I have no idea why. I'm attaching two debug logs:
1) illustrates how it temporarily gets stuck at that file
2) illustrates an invaild pointer operation that arose after performing around 10 sync operations (around line 55600)

I'm not sure if this is a bug, but figured it might be worth checking out.
TagsNo tags attached.
Fixed in build2331

Relationships

related to 0010803 closedLudek MMW v4 Playback of files that needs transcoding is broken in 1635 (regression) 
related to 0017704 closedLudek MMW v4 Invalid pointer on sync 

Activities

Ludek

2021-03-26 11:55

developer   ~0062633

Last edited: 2021-03-26 12:12

1) Based on the log I can confirm that copying the file 'Dead bodies' (2 GB) took 90 seconds.
I guess this is expected amount of time for such a large file, I am just wondering: Are you saying that the sync progress bar isn't updating while copying?
For me (when I send a large file to a device) the progress bar on the bottom is not stucked, but shows the copying progress continually.

Otherwise Michal indicated that the mpc file might be corrupted in the past due to this bug: 0014642

2) I see the "Invalid pointer operation" for thread [10860] that encoded this file:
D:\My Documents\My Music\Air\The Virgin Suicides\10 - Air - Ghost Song.mpc -> D:\PortableApps\MediaMonkey 5\Portable\Temp\MM_AutoConvertedTracks\16FD_10 - Air - Ghost Song.mpcD4E0
but seeing that actually two threads started this file conversion at the same time! Also the thread [8360]. This is unexpected, verifying the code to see how this could happen...
I also wonder -- the crash report hasn't been automatically generated?

Could you upload the '10 - Air - Ghost Song.mpc ' for analyzing? And could you please also verify that the file '10 - Air - Ghost Song.mpc' isn't twice in your library (with the same path) as a possible cause for the "double-conversion" issue mentioned above ?
I suppose that if you re-send just this particular mpc to the device then the 'invalid pointer operation' no longer appears and is related to the "double-conversion" issue mentioned above, right ?

Ludek

2021-03-26 12:51

developer   ~0062635

Last edited: 2021-03-26 12:55

2) I've just found a code glitch that can cause the "double-conversion" issue, was introduced in SVN revision 17858 while fixing issue 0010803 (so 8 years old regression)
=> Fixed in 2331 -- we might want to create clone for merging to MM4

rusty

2021-03-26 13:34

administrator   ~0062637

Last edited: 2021-03-26 13:45

I checked my library and there's only one copy of Ghost Song.

EDIT: Also, fyi, syncing ghost song does _not_ trigger the invalid pointer error.

Ludek

2021-03-26 18:11

developer   ~0062643

ok, so it is fixed in 2331

MM4 clone fixed as 0017704

peke

2021-04-07 21:38

developer   ~0062739

Verified 2333

Unable to replicate in latest version with supplied File and with my set of various test file formats to discover regressions.