Sept. 8, 2014
6:08 p.m.
On Mon, 08 Sep 2014 06:36:34 -0400, nick said:
Found a bug and attempted to fix it sub_skb = dev_alloc_skb(nSubframe_Length + 12); + if (!sub_skb) + return -ENOMEM;
Nick - we've told you before to research this stuff more fully before posting patches. As others have pointed out, there's exactly one caller, who wants a different return on error. For bonus points - explain why you're returning a -ENOMEM from a function that's declared as 'static u8 parse_subframe(...'. This is *NOT* how you convince us that you should be allowed anywhere near kernel code.