Added issue https://github.com/gnuradio/gnuradio/issues/1554

On Sun, Jan 21, 2018 at 7:38 AM, Jeff Long <willco...@gmail.com> wrote:

> Ah, good. I didn't think it needed another +1 but couldn't think of any
> other overflow.
>
> On Sun, Jan 21, 2018 at 7:33 AM, Jeff Long <willco...@gmail.com> wrote:
>
>> The forecast function is supposed to guarantee that there is enough input, 
>> but it might be rounding down in some cases. Could you try adding the "+1" 
>> to the last line and see if that changes anything?
>>
>>
>> void
>> pfb_clock_sync_ccf_impl::forecast(int noutput_items,
>>                                   gr_vector_int &ninput_items_required)
>> {
>>   unsigned ninputs = ninput_items_required.size ();
>>   for(unsigned i = 0; i < ninputs; i++)
>>     ninput_items_required[i] = (noutput_items + history()) * (d_sps/d_osps) 
>> + 1;
>> }
>>
>>
>>
>> On Sun, Jan 21, 2018 at 6:35 AM, Daniel Estévez <dan...@destevez.net>
>> wrote:
>>
>>> El 21/01/18 a las 11:39, Daniel Estévez escribió:
>>> >
>>> > out[i+d_out_idx] = d_filters[d_filtnum]->filter(&in[count+d_out_idx]);
>>> >
>>>
>>> Hi all,
>>>
>>> I've been looking at this again and the problem is that in this line,
>>>
>>> count = -134217704
>>>
>>> for some weird reason. Obviously, this causes the SIGSEGV.
>>>
>>> I'll try to see why count gets corrupted.
>>>
>>> Regards,
>>>
>>> Daniel.
>>>
>>> _______________________________________________
>>> Discuss-gnuradio mailing list
>>> Discuss-gnuradio@gnu.org
>>> https://lists.gnu.org/mailman/listinfo/discuss-gnuradio
>>>
>>
>>
>
_______________________________________________
Discuss-gnuradio mailing list
Discuss-gnuradio@gnu.org
https://lists.gnu.org/mailman/listinfo/discuss-gnuradio

Reply via email to