Csound Csound-dev Csound-tekno Search About

[Csnd-dev] Crashing bug

Date2017-02-18 21:52
FromSteven Yi
Subject[Csnd-dev] Crashing bug
Hi All,

>From Anton's email on the user's list, I experienced a crash that
Hlodver also experienced.  I did a git bisect session and found that
the crashing was first introduced in this commit:

2064211c149fa24dcce9be6b894a97ef93fa8bc8 is the first bad commit
commit 2064211c149fa24dcce9be6b894a97ef93fa8bc8
Author: jpff 
Date:   Mon Feb 6 21:21:10 2017 +0000

    fix score r without macro

The code is not an area I've looked at before.  (A diff of the commit
is here: https://github.com/csound/csound/commit/2064211c149fa24dcce9be6b894a97ef93fa8bc8).

John: could you take a look at this change?  The test file used is
"dafunk-lead.csd" that Anton posted.

Thanks!

Date2017-02-18 22:17
FromSteven Yi
SubjectRe: [Csnd-dev] Crashing bug
BTW: I found out that the issue is rooted in this line of the CSD:

f2 0 8192 10  1.0 0.0 0.0 0.0 5.0e-2

Apparently after that change, the e in the scientific notation leads
to a crash.  Changing that to ".005" manually allows the CSD to run.

On Sat, Feb 18, 2017 at 4:52 PM, Steven Yi  wrote:
> Hi All,
>
> From Anton's email on the user's list, I experienced a crash that
> Hlodver also experienced.  I did a git bisect session and found that
> the crashing was first introduced in this commit:
>
> 2064211c149fa24dcce9be6b894a97ef93fa8bc8 is the first bad commit
> commit 2064211c149fa24dcce9be6b894a97ef93fa8bc8
> Author: jpff 
> Date:   Mon Feb 6 21:21:10 2017 +0000
>
>     fix score r without macro
>
> The code is not an area I've looked at before.  (A diff of the commit
> is here: https://github.com/csound/csound/commit/2064211c149fa24dcce9be6b894a97ef93fa8bc8).
>
> John: could you take a look at this change?  The test file used is
> "dafunk-lead.csd" that Anton posted.
>
> Thanks!

Date2017-02-18 22:21
FromSteven Yi
SubjectRe: [Csnd-dev] Crashing bug
Attachmentstest.csd  
Here is a minimal test.csd that crashes.  We should add this to the
test suite once the code is working again.

On Sat, Feb 18, 2017 at 5:17 PM, Steven Yi  wrote:
> BTW: I found out that the issue is rooted in this line of the CSD:
>
> f2 0 8192 10  1.0 0.0 0.0 0.0 5.0e-2
>
> Apparently after that change, the e in the scientific notation leads
> to a crash.  Changing that to ".005" manually allows the CSD to run.
>
> On Sat, Feb 18, 2017 at 4:52 PM, Steven Yi  wrote:
>> Hi All,
>>
>> From Anton's email on the user's list, I experienced a crash that
>> Hlodver also experienced.  I did a git bisect session and found that
>> the crashing was first introduced in this commit:
>>
>> 2064211c149fa24dcce9be6b894a97ef93fa8bc8 is the first bad commit
>> commit 2064211c149fa24dcce9be6b894a97ef93fa8bc8
>> Author: jpff 
>> Date:   Mon Feb 6 21:21:10 2017 +0000
>>
>>     fix score r without macro
>>
>> The code is not an area I've looked at before.  (A diff of the commit
>> is here: https://github.com/csound/csound/commit/2064211c149fa24dcce9be6b894a97ef93fa8bc8).
>>
>> John: could you take a look at this change?  The test file used is
>> "dafunk-lead.csd" that Anton posted.
>>
>> Thanks!
>> steven

Date2017-02-18 23:08
Fromjpff
SubjectRe: [Csnd-dev] Crashing bug
OK I understad -- too late tonigt toug.  Will attack in morning

On Sat, 18 Feb 2017, Steven Yi wrote:

> Here is a minimal test.csd that crashes.  We should add this to the
> test suite once the code is working again.
>
> On Sat, Feb 18, 2017 at 5:17 PM, Steven Yi  wrote:
>> BTW: I found out that the issue is rooted in this line of the CSD:
>>
>> f2 0 8192 10  1.0 0.0 0.0 0.0 5.0e-2
>>
>> Apparently after that change, the e in the scientific notation leads
>> to a crash.  Changing that to ".005" manually allows the CSD to run.
>>
>> On Sat, Feb 18, 2017 at 4:52 PM, Steven Yi  wrote:
>>> Hi All,
>>>
>>> From Anton's email on the user's list, I experienced a crash that
>>> Hlodver also experienced.  I did a git bisect session and found that
>>> the crashing was first introduced in this commit:
>>>
>>> 2064211c149fa24dcce9be6b894a97ef93fa8bc8 is the first bad commit
>>> commit 2064211c149fa24dcce9be6b894a97ef93fa8bc8
>>> Author: jpff 
>>> Date:   Mon Feb 6 21:21:10 2017 +0000
>>>
>>>     fix score r without macro
>>>
>>> The code is not an area I've looked at before.  (A diff of the commit
>>> is here: https://github.com/csound/csound/commit/2064211c149fa24dcce9be6b894a97ef93fa8bc8).
>>>
>>> John: could you take a look at this change?  The test file used is
>>> "dafunk-lead.csd" that Anton posted.
>>>
>>> Thanks!
>>> steven