On Mon, Jul 27, 2015 at 09:41:10PM -0700, Joshua Clayton wrote: > r8712_get_ndis_wlan_bssid_ex_sz has a "6 * sizeof(unsigned long)" > where the underlying struct has a 6 * unsigned char. > Simplify the calculation by just subtracting the variable part from > the size of the struct. > > This also gets rid of a use of typedef NDIS_802_11_RATES_EX > > Signed-off-by: Joshua Clayton <stillcompiling@xxxxxxxxx> > --- <snip> > > u8 *r8712_get_capability_from_ie(u8 *ie) > -- > 2.4.6 > > From: Joshua Clayton <stillcompiling@xxxxxxxxx> > To: Larry Finger <Larry.Finger@xxxxxxxxxxxx>, > Florian Schilhabel <florian.c.schilhabel@xxxxxxxxxxxxxx>, > Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, > Sudip Mukherjee <sudipm.mukherjee@xxxxxxxxx>, > Nitin Kuppelur <nitinkuppelur@xxxxxxxxx>, > Joshua Clayton <stillcompiling@xxxxxxxxx>, > Vaishali Thakkar <vthakkar1994@xxxxxxxxx>, > Tapasweni Pathak <tapaswenipathak@xxxxxxxxx>, > Daniel Baluta <daniel.baluta@xxxxxxxxx>, > Melike Yurtoglu <aysemelikeyurtoglu@xxxxxxxxx>, > Max Perepelitsyn <mperepelitsyn@xxxxxxxxx>, > Aya Mahfouz <mahfouz.saif.elyazal@xxxxxxxxx>, > Cristina Opriceana <cristina.opriceana@xxxxxxxxx>, > Dogukan Ergun <dogukan.ergun@xxxxxxxxx>, > Julia Lawall <Julia.Lawall@xxxxxxx>, > Dan Carpenter <dan.carpenter@xxxxxxxxxx>, > Haneen Mohammed <hamohammed.sa@xxxxxxxxx>, > Rickard Strandqvist <rickard_strandqvist@xxxxxxxxxxxxxxxxxx> > Cc: devel@xxxxxxxxxxxxxxxxxxxx, > linux-kernel@xxxxxxxxxxxxxxx > Subject: [PATCH V3 1/8] staging: rtl8712: fix buggy size calculation > Date: Mon, 27 Jul 2015 21:41:10 -0700 > Message-Id: <305999598d7738feabdc8e5df5400b3e7be7f5ed.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > In-Reply-To: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > References: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > In-Reply-To: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > References: <cover.1438057939.git.stillcompiling@xxxxxxxxx> Something is wrong. The same patch is getting repeated in the same mail multiple times. I deleted almost 2500 lines after this before sending this reply. See how your patch is looking like: http://www.gossamer-threads.com/lists/linux/kernel/2224869 regards sudip _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
