bugGNU Octave - Bugs: bug #41163, Octave crashes when compiled with...

 
 

bug #41163: Octave crashes when compiled with 64 bit indexing and jit enabled

Submitted by:  Lasse Schuirmann <sils1297>
Submitted on:  Thu 09 Jan 2014 12:09:20 PM UTC  
 
Category: Configuration and Build SystemSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Build Failure
Status: FixedAssigned to: None
Originator Name: Lasse SchuirmannOpen/Closed: Closed
Release: devOperating System: GNU/Linux

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Tue 14 Jan 2014 07:28:05 PM UTC, comment #4:

No worries about the patch. I simplified both instances and checked in a patch (http://hg.savannah.gnu.org/hgweb/octave/rev/e243b705a726), giving you credit since you saw this first.

Rik <rik5>
Project Administrator
Tue 14 Jan 2014 07:10:48 PM UTC, comment #3:

There is no reason besides me trying to mimic the code style in this document. The same case is handled this way in line 1613 so I just copied it.

So I can upload a simplified patch tomorrow or so.

Lasse Schuirmann <sils1297>
Tue 14 Jan 2014 06:52:39 PM UTC, comment #2:

The attached patch is written as

Is there a reason why one couldn't just declare the variable to be int in the first place and avoid the if statement?

The only other place in the block that uses the variable "one" is

which also has an int object as the first one for comparison.

Rik <rik5>
Project Administrator
Fri 10 Jan 2014 09:30:45 PM UTC, comment #1:

@Lasse: could you please post a diff of your fix? Thanks.

Felipe G. Nievinski <fgnievinski>
Thu 09 Jan 2014 12:09:20 PM UTC, original submission:

When compiling with 64 bit indexing and jit octave crashes at runtime (compilation succeded with the patch from https://savannah.gnu.org/bugs/?41145):

I found that the bug was in libinterp/corefcn/jit-typeinfo.cc:
while one->getType() returns "i64" and rcount->getType() returns "i32". Therefore I created an ione in this scope (as above in line 1613) to match the type of rcond.

Best,

Lasse

Lasse Schuirmann <sils1297>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #30233:  jit_64index.patch added by sils1297 (1KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by rik5 (Posted a comment)
  • -unavailable- added by mtmiller (Updated the item)
  • -unavailable- added by fgnievinski (Posted a comment)
  • -unavailable- added by sils1297 (Submitted the item)
  • -unavailable- added by sils1297
  •  

    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
    Tue 14 Jan 2014 07:28:05 PM UTCrik5StatusPatch Submitted=>Fixed
      Open/ClosedOpen=>Closed
    Sat 11 Jan 2014 03:57:59 AM UTCmtmillerStatusNone=>Patch Submitted
    Thu 09 Jan 2014 05:30:17 PM UTCrik5Carbon-CopyRemoved 72865=>-
    Thu 09 Jan 2014 05:30:02 PM UTCrik5CategoryInterpreter=>Configuration and Build System
      Item GroupSegfault, Bus Error, etc.=>Build Failure
    Thu 09 Jan 2014 12:09:20 PM UTCsils1297Attached File-=>Added jit_64index.patch, #30233
      Carbon-Copy-=>Added -unavailable-

    Back to the top


    Powered by Savane 3.1-cleanup1