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

 
 

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

Submitted by:  PrasannaKumar Muralidharan <prasannatsm>
Submitted on:  Wed 08 Oct 2014 05:30:41 PM UTC  
 
Category: Octave FunctionSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: devOperating System: Any

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

(Jump to the original submission 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>
Project 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>
Project 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>
Project Member
Thu 09 Oct 2014 12:30:11 PM UTC, comment #1:

Duplicate of #43384

Ben Abbott <bpabbott>
Project 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 File(s):
   
   
Comment:
   

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jwe (Posted a comment)
  • -unavailable- added by bpabbott (Posted a comment)
  • -unavailable- added by prasannatsm (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 8 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 09 Oct 2014 09:13:35 PM UTCjweStatusNone=>Fixed
      Open/ClosedOpen=>Closed
    Thu 09 Oct 2014 05:30:45 PM UTCbpabbottStatusDuplicate=>None
      Open/ClosedClosed=>Open
      SummaryFix using uninitialised variable=>Fix using uninitialized variable (_ilu_.cc)
    Thu 09 Oct 2014 12:30:11 PM UTCbpabbottStatusNone=>Duplicate
      Open/ClosedOpen=>Closed
    Wed 08 Oct 2014 05:30:41 PM UTCprasannatsmAttached File-=>Added fix_uninitialised_memory_access_-__ilu__-_cc.diff, #32243

    Back to the top


    Powered by Savane 3.1-cleanup1