bugPSPP - Bugs: bug #48128, FREQUENCIES crashes with...

 
 

bug #48128: FREQUENCIES crashes with combination of /statistics /percentiles and /histogram options

Submitted by:  Friedrich Beckmann <beckmanf>
Submitted on:  Sat 04 Jun 2016 04:16:38 PM UTC  
 
Category: Syntax ParserSeverity: 5 - Average
Status: FixedAssigned to: None
Open/Closed: ClosedRelease: None
Effort: 0.00

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Sat 18 Jun 2016 07:44:08 AM UTC, comment #5:

I think this is fixed now.

John Darrington <jmd>
Project Administrator
Wed 08 Jun 2016 06:29:17 PM UTC, comment #4:

I added a test to the regression for this situation with commit

http://git.savannah.gnu.org/cgit/pspp.git/commit/?id=de660f95574

Friedrich Beckmann <beckmanf>
Project Member
Mon 06 Jun 2016 04:30:19 PM UTC, comment #3:

Hi John,

yes, maybe. I only tested in the gui, but it is not gui related. I will add a crash test.

Friedrich

Friedrich Beckmann <beckmanf>
Project Member
Mon 06 Jun 2016 04:22:40 PM UTC, comment #2:

Thanks for fixing this.

Do you think it would be a good idea to add a test to frequencies.at for this scenario?

John Darrington <jmd>
Project Administrator
Sat 04 Jun 2016 10:52:53 PM UTC, comment #1:

I pushed a fix for this:

http://git.savannah.gnu.org/cgit/pspp.git/commit/?id=86642e66638412248901e54a4392882c32d83a71

The 0.75 percentiles with attribute show=false for the histogram appeared as show=true after this removal of double entries code section. The number of n_show_percentiles did not fit to this.

Friedrich Beckmann <beckmanf>
Project Member
Sat 04 Jun 2016 04:16:38 PM UTC, original submission:

Cubyrk Zyrvay reported the following bug on the mailing list:

http://lists.gnu.org/archive/html/bug-gnu-pspp/2016-06/msg00000.html

To produce the crash, run the following syntax:

This will result in

Cubyrk reported this bug for linux. This bug is also visible on MacOS in the current git version. If one of the options

/STATISTICS
/PERCENTILES ...
/HISTOGRAM

is removed, no crash happens.

Friedrich Beckmann <beckmanf>
Project Member

 

(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 jmd (Posted a comment)
  • -unavailable- added by beckmanf (Submitted the item)
  • -unavailable- added by beckmanf
  •  

    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 logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 18 Jun 2016 07:44:08 AM UTCjmdOpen/ClosedOpen=>Closed
    Mon 06 Jun 2016 04:22:40 PM UTCjmdOpen/ClosedClosed=>Open
    Sat 04 Jun 2016 10:52:53 PM UTCbeckmanfStatusConfirmed=>Fixed
      Open/ClosedOpen=>Closed
    Sat 04 Jun 2016 04:16:38 PM UTCbeckmanfCarbon-Copy-=>Added -unavailable-

    Back to the top


    Powered by Savane 3.1-cleanup1