[Acx100-devel] fixed memory leak

[<mr...@se...>]

Hi
The following patch:

*** acx100/src/acx100.c 2003-07-27 22:48:50.000000000 +0200
--- acx100.test/src/acx100.c    2003-08-03 19:35:26.000000000 +0200
***************
*** 818,823 ****
--- 818,828 ----
                 txresult = dmaTxData(wlandev, pb);
                 if (txresult == 1){
   //                    acx100_unlock(wlandev,&flags);
+                       if (pb->p80211hostbuf != NULL)
+                               kfree(pb->p80211hostbuf);
+
+                       if (pb != NULL)
+                               kfree(pb);
                         return 1;
                 }
                 acxlog(L_XFER, "wake queue for future Tx.\n");

fixes big memory leak in function acx100_start_xmit - memory leaks after 
unsuccesfull call to dmaTxData (when ring buffer is full). It happends 
quite often during big transfers beetwen wired lan and wireless lan or 
ping floods, every time when "Can't send packet, no txdesc free (yet)!" 
message appears in debug log.

I have been tested this patch little (sending huge amounts of data) - 
everything seems to be ok.

Please include this patch in next driver release, if possible.

-- 
mrk(at)sed.pl