Bugs item #3088391, was opened at 2010-10-15 23:35
Message generated for change (Comment added) made by veplaini
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: Open
Resolution: None
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 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

------------------------------------------------------------------------------
Download new Adobe(R) Flash(R) Builder(TM) 4
The new Adobe(R) Flex(R) 4 and Flash(R) Builder(TM) 4 (formerly 
Flex(R) Builder(TM)) enable the development of rich applications that run
across multiple browsers and platforms. Download your free trials today!
http://p.sf.net/sfu/adobe-dev2dev
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net