bugGNU Octave - Bugs: bug #40513, the line '##{' eats memory and cpu

 
 

bug #40513: the line '##{' eats memory and cpu

Submitted by:  Muhali <muhali>
Submitted on:  Thu 07 Nov 2013 01:57:17 PM UTC  
 
Category: InterpreterSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Segfault, Bus Error, etc.
Status: FixedAssigned to: John W. Eaton <jwe>
Originator Name: Open/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.

 

(Jump to the original submission Jump to the original submission)

Thu 07 Nov 2013 11:29:45 PM UTC, comment #9:

Good news. I built and installed a current
system. Now, ##{
returns prompt. No hang.
Before, I just ran
hg update
make

So, works for me!

Michael Godfrey <godfrey>
Project Member
Thu 07 Nov 2013 10:58:30 PM UTC, comment #8:

Or try "__lexer_debug_flag__ (true)" before trying to execute foo.m. That should tell you the patterns that the lexer is matching. Previously, it was matching line comment but then (incorrectly) detecting the start of a block comment, then trying again and instead of matching a block comment, it matched a line comment again. And so on, with no end. If it is still doing that for you, then I suspect that something happened so that the copy of Octave you are running does not have the fix I checked in.

John W. Eaton <jwe>
Project AdministratorIn charge of this item.
Thu 07 Nov 2013 10:47:02 PM UTC, comment #7:

I'll re-open the report. I think you might need to run under gdb and figure out what loop it is getting caught in.

Rik <rik5>
Project Administrator
Thu 07 Nov 2013 09:54:22 PM UTC, comment #6:

Well, Rik's experience was better than mine.
Takes 2 ctrl-C's now.

Michael Godfrey <godfrey>
Project Member
Thu 07 Nov 2013 08:44:58 PM UTC, comment #5:

Works for me. Closing report.

Rik <rik5>
Project Administrator
Thu 07 Nov 2013 08:29:13 PM UTC, comment #4:

I checked in the following changeset:

http://hg.savannah.gnu.org/hgweb/octave/rev/cf3f828f401c

John W. Eaton <jwe>
Project AdministratorIn charge of this item.
Thu 07 Nov 2013 05:49:03 PM UTC, comment #3:

I'll take a look at this problem.

John W. Eaton <jwe>
Project AdministratorIn charge of this item.
Thu 07 Nov 2013 04:36:59 PM UTC, comment #2:

Since this requires ctrl_C to kill Octave
it should be in the "crash: list.

Michael Godfrey <godfrey>
Project Member
Thu 07 Nov 2013 04:28:14 PM UTC, comment #1:

Confirmed.

The parser is probably looking for the end of the block comment and not finding it and then spinning needlessly.

Rik <rik5>
Project Administrator
Thu 07 Nov 2013 01:57:17 PM UTC, original submission:

If foo.m contains the single line

executing 'octave foo.m' eats up memory and cpu ad infinitum.

Muhali <muhali>

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

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 godfrey (Posted a comment)
  • -unavailable- added by rik5 (Posted a comment)
  • -unavailable- added by rik5
  • -unavailable- added by muhali (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 10 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 07 Nov 2013 11:36:09 PM UTCrik5StatusIn Progress=>Fixed
      Open/ClosedOpen=>Closed
    Thu 07 Nov 2013 10:47:13 PM UTCrik5Open/ClosedClosed=>Open
    Thu 07 Nov 2013 10:47:02 PM UTCrik5StatusFixed=>In Progress
    Thu 07 Nov 2013 08:44:58 PM UTCrik5StatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Thu 07 Nov 2013 08:32:06 PM UTCjweStatusConfirmed=>Ready For Test
    Thu 07 Nov 2013 05:49:03 PM UTCjweAssigned toNone=>jwe
    Thu 07 Nov 2013 04:28:14 PM UTCrik5Carbon-Copy-=>Added jwe
    Thu 07 Nov 2013 04:28:13 PM UTCrik5StatusNone=>Confirmed

    Back to the top


    Powered by Savane 3.1-cleanup1