xpra icon
Bug tracker and wiki

Opened 5 years ago

Closed 3 years ago

Last modified 2 years ago

#628 closed defect (wontfix)

ffmpeg + gcc 4.8.x dec_avcodec compatibility on win32

Reported by: Antoine Martin Owned by: Antoine Martin
Priority: major Milestone: 2.0
Component: platforms Version: trunk
Keywords: win32 Cc:

Description

Despite compiling without warnings against both ffmpeg 2.3 and 2.3.1, the resulting dec_avcodec2 video decoder module causes cleanup errors on exit on win32. (that look like use after free or memory corruption: memory accesses at low addresses like 0x00000004)

I think it is safer to go with the 2.2.x branch (2.2.6 is out), even on OSX and Linux: done in r7175.
Please confirm that the problem exists with 2.3 on win32 and re-assign to me as non-blocker for 0.15.

Change History (7)

comment:1 Changed 5 years ago by Antoine Martin

Note: may not be a bug, was seeing the same thing with 2.2.6, a good old reboot-windows later and it's gone..

Found some xpra cleanup bugs, fixed in r7199 (will backport).

comment:2 Changed 5 years ago by Antoine Martin

Well, well, well.

There is a problem, but it is not with the ffmpeg version.
I made a fresh 2.2.5 build (using the exact same command line) and it exhibits the same problem at runtime, the "old" 2.2.5 one does not.

Diffing the two trees shows absolutely nothing of interest, the only files shown as being different are the ".lib", ".a" and ".dll" - which is kind of expected.

So it must be a toolchain related bug. GCC got updated to 4.8.x in mingw32 recently (mingw-get update;mingw-get upgrade...).
It could be a bug in GCC or maybe the newer GCC turns on some stricter checks in the ffmpeg code.
Disabling w32threads makes no difference.

So, now we downgrade GCC back to 4.7, except the downgrade breaks mingw...
You have to download the few dlls that go MIA yourself from here: http://sourceforge.net/projects/mingw/files/MinGW/Base/.
And miracle, it no longer crashes.

comment:3 Changed 5 years ago by Antoine Martin

Keywords: win32 added
Milestone: 0.140.15
Owner: changed from Smo to Antoine Martin
Priority: blockermajor
Status: newassigned
Summary: ffmpeg 2.3.x dec_avcodec compatibilityffmpeg + gcc 4.8.x dec_avcodec compatibility on win32

(updating bug title)

comment:4 Changed 4 years ago by Antoine Martin

Milestone: 0.15future

I don't have time for toolchain issues.

comment:5 Changed 3 years ago by Antoine Martin

FWIW: tried again today and it still causes crashes. Either building ffmpeg with gcc 4.8, or building the cython modules that link to it triggers crashes.

comment:6 Changed 3 years ago by Antoine Martin

Resolution: wontfix
Status: assignedclosed

Superseded by #678.

Closing as wontfix: this build system will remain as it is for now, not updating gcc.

comment:7 Changed 2 years ago by Antoine Martin

Milestone: future2.0
Note: See TracTickets for help on using tickets.