Bug#721147: Potential fix (state: works for me)

2018-10-12 Thread Tzafrir Cohen
On 12/10/18 09:41, Petter Reinholdtsen wrote: > Control: tags -1 + patch > > [Karsten Richter 2015-02-13] >> Patch: >> >> 1. Replace the struct sk_buff with a plain void * that is only allocated >> when needed. It’s a throw away buffer, so no need the added complexity of >> sk_buff. >> 2.

Bug#721147: Potential fix (state: works for me)

2018-10-12 Thread Petter Reinholdtsen
Control: tags -1 + patch [Karsten Richter 2015-02-13] > Patch: > > 1. Replace the struct sk_buff with a plain void * that is only allocated when > needed. It’s a throw away buffer, so no need the added complexity of sk_buff. > 2. Remove memcpy which copies frame data from the channel buffer to

Bug#721147: Potential fix (state: works for me)

2015-02-13 Thread Karsten Richter
Hi, I also experienced this bug on my machine (bug is still present in latest dahdi-linux code in SID), so I investigated a bit into the code. Analysis: I recognized that the sk_buff at first sight is needed in both cases of the if statement, but the memcpy to the sk_buff in the true case