Menu

#5513 Avoid some crashes for bad "control-points" property

Fixed
Enhancement
2019-05-06
2019-04-25
No

Avoid some crashes for bad "control-points" property

http://codereview.appspot.com/576610043

Discussion

  • David Kastrup

    David Kastrup - 2019-04-25
    • Description has changed:

    Diff:

    
    
    • Needs: -->
    • Type: Enhancement --> Crash
     
  • Anonymous

    Anonymous - 2019-04-26
    • Patch: new --> review
     
  • Anonymous

    Anonymous - 2019-04-26

    Passes make, make check and a full make doc.

     
  • Anonymous

    Anonymous - 2019-04-26

    I wonder if it would be useful to put in a warning message if someone 'by mistake' does this in their .ly file? (e.g. a kind of warning against an 'April in the the Spring' mistake)?

     
  • Thomas Morley

    Thomas Morley - 2019-04-26

    There's a recent discussion on -devel about how to draw curves with more than 4 control-points.
    http://lilypond.1069038.n5.nabble.com/stack-smashing-detected-tt221627.html
    Not sure yet what to do and how ..., but I'd love to keep the possibility of entering more than the default amount of control-point without a type-checking warning here.
    Ofcourse ly:slur::print, etc needs 4 control-points not more or less.
    Thus a warning while applying ly:slur::print to a not suitable amount of control-points could warn. (More control-points will need a different or extended procedure.)
    Is this doable with a reasonable amount of work?

     
    • David Kastrup

      David Kastrup - 2019-04-26

      "Thomas Morley" thomas-morley@users.sourceforge.net writes:

      There's a recent discussion on -devel about how to draw curves with
      more than 4 control-points.
      http://lilypond.1069038.n5.nabble.com/stack-smashing-detected-tt221627.html
      Not sure yet what to do and how ..., but I'd love to keep the
      possibility of entering more than the default amount of control-point
      without a type-checking warning here.
      Ofcourse ly:slur::print, etc needs 4 control-points not more or less.
      Thus a warning while applying ly:slur::print to a not suitable amount
      of control-points could warn. (More control-points will need a
      different or extended procedure.)
      Is this doable with a reasonable amount of work?

      I don't think it's worth the trouble: there are so many other ways to
      provide faulty data. This issue is about avoiding clobbering the stack,
      not more, not less. One could let it bomb out with some type error
      instead by being less accepting, but adding additional code/warnings
      seems to be mostly useless since most other ways to mess up would not be
      covered anyway.

      --
      David Kastrup

       
  • Anonymous

    Anonymous - 2019-04-27
    • Patch: review --> countdown
     
  • Anonymous

    Anonymous - 2019-04-27

    Patch on countdown for April 30th.

     
  • Anonymous

    Anonymous - 2019-04-28
    • Needs: -->
    • Patch: new --> review
    • Type: --> Enhancement
     
  • Anonymous

    Anonymous - 2019-04-28

    Passes make, make check and a full make doc.

     
  • Anonymous

    Anonymous - 2019-05-01
    • Patch: review --> countdown
     
  • Anonymous

    Anonymous - 2019-05-01

    Patch on countdown for May 3rd

     
  • Anonymous

    Anonymous - 2019-05-03

    Patch counted down - please push to staging.

     
  • Anonymous

    Anonymous - 2019-05-03
    • Patch: countdown --> push
     
  • Anonymous

    Anonymous - 2019-05-06
    • labels: --> Fixed_2_21_0
    • status: Started --> Fixed
    • Patch: push -->
     
  • Anonymous

    Anonymous - 2019-05-06
    Issue 5513: Avoid some crashes for bad "control-points" property
    author  David Kastrup <dak@gnu.org> 
        Thu, 25 Apr 2019 17:33:59 +0000 (19:33 +0200)
    committer   David Kastrup <dak@gnu.org> 
        Fri, 3 May 2019 10:08:44 +0000 (12:08 +0200)
    commit  ab8e7e4d89a4b2a6b440220c9fd6e80b4a1f0399