bugGNU Octave - Bugs: bug #43384, Fix using uninitialized variable...

 
 

bug #43384: Fix using uninitialized variable (__ilu__.cc)

Submitter:  PrasannaKumar Muralidharan <prasannatsm>
Submitted:  Wed 08 Oct 2014 05:30:41 PM UTC
   
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Thu 09 Oct 2014 09:13:35 PM UTC, comment #6: 

I applied this changeset.  Thanks for the patch.

Although this function is not supposed to be called directly, it is possible to do so, and in that case it appears to me that it could be called in a way that would leave these variables uninitialized.

John W. Eaton <jwe>
Group administrator
Thu 09 Oct 2014 06:07:08 PM UTC, comment #5: 

I have not tried that yet. I am running Coverity static analyser on Octave, it found this issue.

Based on my past experience the code works somehow even if some variables are used without inititalising so unfortunately even a test case may not be able to uncover it.

PrasannaKumar Muralidharan <prasannatsm>
Thu 09 Oct 2014 05:30:45 PM UTC, comment #4: 

Ok. My mistake. Please include additional information.  Is this bug something that is detectable from the command line?

Ben Abbott <bpabbott>
Group Member
Thu 09 Oct 2014 03:56:31 PM UTC, comment #3: 

Both patch fix same issue but in different files. Should I prepare a single patch?

The patch attached with this bug should still be useful.

PrasannaKumar Muralidharan <prasannatsm>
Thu 09 Oct 2014 12:34:59 PM UTC, comment #2: 

opps ... Duplicate of #43383

Ben Abbott <bpabbott>
Group Member
Thu 09 Oct 2014 12:30:11 PM UTC, comment #1: 

Duplicate of #43384

Ben Abbott <bpabbott>
Group Member
Wed 08 Oct 2014 05:30:41 PM UTC, original submission:  

Attached patch fixes use of uninitialised variable.

PrasannaKumar Muralidharan <prasannatsm>

 

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

Attach Files:
   
   
Comment:
   

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by jwe (Posted a comment)
  • -email is unavailable- added by bpabbott (Posted a comment)
  • -email is unavailable- added by prasannatsm (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-10-09 jwe StatusNone Fixed
        Open/ClosedOpen Closed
    2014-10-09 bpabbott StatusDuplicate None
        Open/ClosedClosed Open
        SummaryFix using uninitialised variable Fix using uninitialized variable (_ilu_.cc)
    2014-10-09 bpabbott StatusNone Duplicate
        Open/ClosedOpen Closed
    2014-10-08 prasannatsm Attached File- Added fix_uninitialised_memory_access_-__ilu__-_cc.diff, #32243

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code