bugGNU Octave - Bugs: bug #51057, [octave forge] video package...


bug #51057: [octave forge] video package crashes using aviinfo

Submitted by:  John Donoghue <lostbard>
Submitted on:  Thu 18 May 2017 01:28:11 PM UTC  
Category: Octave Forge PackageSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Segfault, Bus Error, etc.
Status: FixedAssigned to: None
Originator Name: JohnDOpen/Closed: Closed
Release: 4.2.1Operating System: Microsoft Windows

Add a New Comment(Rich Markup)

You are not logged in

Please log in, so followups can be emailed to you.


Thu 01 Jun 2017 02:14:39 PM UTC, comment #4:

works with new release of video 1.2.4 so closing as fixed

John Donoghue <lostbard>
Project Member
Fri 19 May 2017 07:52:22 PM UTC, comment #3:

Thank you John, for debugging and fixing this issue.

I've added your patch and requested a bugfix release https://sourceforge.net/p/octave/package-releases/308/

change status to "ready for test"

Andreas Weber <andy1978>
Project Member
Fri 19 May 2017 03:55:46 PM UTC, comment #2:

Ok so issue is actually the get_filedate. If I change the localtime call to ::localtime, it works ok.

In addition its creating a new temp buffer to strftime with that is never freed.

Since octave has functions to do the local time and strftime, I have attached a patch that changes it to do that - and it no longer crashes in windows.

(file #40737)

John Donoghue <lostbard>
Project Member
Thu 18 May 2017 01:37:06 PM UTC, comment #1:

I tried a reinstall of the video package for forge to see if it was an issue with cross compiling the video package, however it does the same thing with the reinstalled version as well.

John Donoghue <lostbard>
Project Member
Thu 18 May 2017 01:28:11 PM UTC, original submission:

Using the video package on a avi file in windows 10, calling

crashes octave on a both an existing file and non existing file

For a source file, I did a search for test.avi in google which came up with this link: http://www.engr.colostate.edu/me/facil/dynamics/files/flame.avi

With a quick debug, with no symbols gdb reports:

John Donoghue <lostbard>
Project Member


(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):

Attached Files
file #40737:  bug51057.patch added by lostbard (1KiB - application/octet-stream)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by andy1978 (Posted a comment)
  • -unavailable- added by lostbard (Adding package maintainer)
  • -unavailable- added by lostbard (Submitted the item)

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.


    Please enter the title of George Orwell's famous dystopian book (it's a date):



    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 01 Jun 2017 02:14:39 PM UTClostbardStatusReady For Test=>Fixed
    Fri 19 May 2017 07:52:22 PM UTCandy1978StatusPatch Submitted=>Ready For Test
    Fri 19 May 2017 03:56:52 PM UTClostbardCarbon-Copy-=>Added andy1978
    Fri 19 May 2017 03:55:46 PM UTClostbardAttached File-=>Added bug51057.patch, #40737
      StatusNone=>Patch Submitted

    Back to the top

    Powered by Savane 3.1-cleanup1