From mboxrd@z Thu Jan  1 00:00:00 1970
From: xerofoify@gmail.com (nick)
Date: Tue, 09 Sep 2014 22:56:10 -0400
Subject: Trial Patch
In-Reply-To: <105811.1410317573@turing-police.cc.vt.edu>
References: <540E6F3E.60802@gmail.com>
	<2c423d93-f6a9-4141-a856-6dce6de56599@email.android.com>
	<CA+MoWDpfPz1UCSOJ6+Jc2vXsTy-hNmotR-L0P2kwP_Kh=z_JUg@mail.gmail.com>
	<CAPDOMVhtYj_R6dfdhEiPMKheXDD1y_WCTNmFsSLRYwP-L=FPUg@mail.gmail.com>
	<59721.1410280833@turing-police.cc.vt.edu>
	<540F6E23.9090508@gmail.com>
	<87938.1410300222@turing-police.cc.vt.edu>
	<540FA78E.5080400@gmail.com>
	<105811.1410317573@turing-police.cc.vt.edu>
Message-ID: <540FBDCA.6010408@gmail.com>
To: kernelnewbies@lists.kernelnewbies.org
List-Id: kernelnewbies.lists.kernelnewbies.org



On 14-09-09 10:52 PM, Valdis.Kletnieks at vt.edu wrote:
> On Tue, 09 Sep 2014 21:21:18 -0400, nick said:
>> Not going to execute, I wasn't sure if that statement needs to be executed.
> 
>  		skb  = dev_alloc_skb(frag_length + 4);
> +		if (skb == NULL) {
> +			skb_queue_purge(&priv->rtllib->skbwaitQ[TXCMD_QUEUE];
> +			break;
> +		}
> 
> OK. It's official.  I see *zero* possibility that you'll ever manage to
> write an acceptable patch.  This is *still* wrong.
> 
Care to explain.
Nick