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 May 18 13:28:11 2017  
 
Category: Octave Forge PackageSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Segfault, Bus Error, etc.
Status: Ready For TestAssigned to: None
Originator Name: JohnDOpen/Closed: Open
Release: 4.2.1Operating System: Microsoft Windows

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

Fri May 19 19:52:22 2017, 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 May 19 15:55:46 2017, 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 May 18 13:37:06 2017, 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 May 18 13:28:11 2017, 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):
   
   
Comment:
   

Attached Files
file #40737:  bug51057.patch added by lostbard (1kB - 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 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri May 19 19:52:22 2017andy1978StatusPatch Submitted=>Ready For Test
    Fri May 19 15:56:52 2017lostbardCarbon-Copy-=>Added andy1978
    Fri May 19 15:55:46 2017lostbardAttached File-=>Added bug51057.patch, #40737
      StatusNone=>Patch Submitted

    Back to the top


    Powered by Savane 3.1-cleanup1