| Bugs item #3088391, was opened at 2010-10-15 23:35
Message generated for change (Settings changed) made by jpff
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=564599&aid=3088391&group_id=81968
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: csound
Group: csound5
>Status: Closed
Resolution: Fixed
Priority: 5
Private: No
Submitted By: Hibou57 (hibou-57)
Assigned to: Nobody/Anonymous (nobody)
Summary: Segmentation violation (fault) on syntax error in CSD file
Initial Comment:
Some syntax errors seems to crash CSound. Here is one in the attached file. At line 48, a comma is missing. Open it in WinXound with CSound5, you will get a segmentation violation. Add the comma where it is missing (after the opcode name) and everything will be all right.
----------------------------------------------------------------------
Comment By: Victor Lazzarini (veplaini)
Date: 2010-10-17 17:58
Message:
NB: fixed with --old-parser, nothing done to new parser
----------------------------------------------------------------------
Comment By: Victor Lazzarini (veplaini)
Date: 2010-10-17 17:58
Message:
I committed a fix to prevent this crash. In CVS now.
----------------------------------------------------------------------
Comment By: John ffitch (jpff)
Date: 2010-10-17 14:01
Message:
I have added mor eerror cases to the new parser but have not yet tested it
----------------------------------------------------------------------
Comment By: Victor Lazzarini (veplaini)
Date: 2010-10-17 08:42
Message:
Ok, if I remove a comma from the Opcode definition, there is an error
followed by a crash. This happens only in this case, other syntax errors
just give an error an exit. Probably requires a jump at the point or
something like that, so to be handled like any other syntax errors. Not
sure this should be fixed in the old parser which will be discontinued, but
certainly one to bear in mind for the new parser.
----------------------------------------------------------------------
Comment By: Hibou57 (hibou-57)
Date: 2010-10-16 12:42
Message:
This is CSound5 Version 5.12.1 on Windows XP SP3. Was downloaded from
SourceForge (was an installer).
----------------------------------------------------------------------
Comment By: Victor Lazzarini (veplaini)
Date: 2010-10-16 09:06
Message:
More details needed? What version? Is this from CVS? Which system?
The latest CVS code enables the new parser by default, which is currenty
producing segmentation faults when it cannot parse an orchestra. If this is
from CVS, it's a known problem, which you can solve (for now) by reverting
to the old parser, using --old-parser
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=564599&aid=3088391&group_id=81968
------------------------------------------------------------------------------
The Next 800 Companies to Lead America's Growth: New Video Whitepaper
David G. Thomson, author of the best-selling book "Blueprint to a
Billion" shares his insights and actions to help propel your
business during the next growth cycle. Listen Now!
http://p.sf.net/sfu/SAP-dev2dev
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net |