[Bug] Fail to import files > 2 Gb [sf#263]

[rbouqueau]

Reported by kurtnoise13 on 2013-01-08 14:05 UTC Hi,

I'm trying to create an mp4 file using this command line :

MP4Box -add in_01.mp4 -cat in_02.h264 -new out.mp4

But MP4Box crashes during importation :

[iso file] Read Box ")..}" failed (Out Of Memory) Assertion failed!

Program: U:\Dev\gpac_svn\bin\gcc\MP4Box.exe File: utils/xml_parser.c, Line 1224

Expression: gf_f64_tell(test) < 1<<31

This application has requested the Runtime to terminate it in an unusual way. Please contact the application's support team for more information.

My in_01.mp4 filesize is > 2Gb.

Using files size <= 2Gb, all it's ok.

[rbouqueau]

Commented by rbouqueau on 2013-01-08 15:20 UTC Hi Kurtnoise,

Can you try with an official binary and tell me if it works? http://gpac.wp.mines-telecom.fr/downloads/gpac-nightly-builds/

MinGW's header were not up to date (and we haven't checked either recently or with mingw-w64). Which toolchain do you use?

Romain

[rbouqueau]

Commented by kurtnoise13 on 2013-01-09 08:47 UTC Hi Romain,

Yes, it works using official builds...So, it seems to be an issue from my own build :-/

I'm using the official MinGW toolsuite. So, I need to change my compiler...

btw, could you commit this patch please ?

Index: applications/mp4box/main.c

--- applications/mp4box/main.c (revision 4266) +++ applications/mp4box/main.c (working copy) @@ -212,6 +213,7 @@ " Note 1: some tracks may be removed in the process\n" " Note 2: always on for .3gp .3g2 *.3gpp\n" " -ipod rewrites the file for iPod\n"

• " -psp rewrites the file for PSP devices\n" " -brand ABCD[:v] sets major brand of file, with optional version\n" " -ab ABCD adds given brand to file's alternate brand list\n" " -rb ABCD removes given brand from file's alternate brand list\n" @@ -1544,6 +1553,7 @@ else if (!stricmp(arg, "-isma")) { conv_type = GF_ISOM_CONV_TYPE_ISMA; open_edit = 1; } else if (!stricmp(arg, "-3gp")) { conv_type = GF_ISOM_CONV_TYPE_3GPP; open_edit = 1; } else if (!stricmp(arg, "-ipod")) { conv_type = GF_ISOM_CONV_TYPE_IPOD; open_edit = 1; }
• else if (!stricmp(arg, "-psp")) { conv_type = GF_ISOM_CONV_TYPE_PSP; open_edit = 1; } else if (!stricmp(arg, "-ismax")) { conv_type = GF_ISOM_CONV_TYPE_ISMA_EX; open_edit = 1; }

else if (!stricmp(arg, "-no-sys") || !stricmp(arg, "-nosys")) { remove_sys_tracks = 1; open_edit = 1; }

[rbouqueau]

Commented by jeanlf on 2013-01-09 09:40 UTC Closing the bug - patch is now on SVN, thanks for it.

[rbouqueau]

Updated by jeanlf on 2013-01-09 09:40 UTC

• status: open --> closed-out-of-date