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 Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Segfault, Bus Error, etc.
Status:  Fixed Assigned to:  None
Originator Name:  JohnD Open/Closed:  Closed
Release:  4.2.1 Operating 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 Files:
   
   
Comment:
   

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
  • -email is unavailable- added by andy1978 (Posted a comment)
  • -email is unavailable- added by lostbard (Adding package maintainer)
  • -email is unavailable- added by lostbard (Submitted the item)
  •  

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

    Only project members can vote.

     

     

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-06-01 lostbard StatusReady For Test => Fixed
        Open/ClosedOpen => Closed
    2017-05-19 andy1978 StatusPatch Submitted => Ready For Test
    2017-05-19 lostbard Carbon-Copy- => Added andy1978
    2017-05-19 lostbard Attached File- => Added bug51057.patch, #40737
        StatusNone => Patch Submitted

    Back to the top


    Powered by Savane 3.3