Something horrible seems to have happened in your patch sending process, and you have the same message over and over. Also, you don't really need to seen the patch to everyone who has ever touched the file. The following arguments can be useful: --nokeywords --nogit --nogit-fallback --norolestats julia On Mon, 27 Jul 2015, Joshua Clayton wrote: > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 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 0/8] clean up wlan_bssdef.h > Date: Mon, 27 Jul 2015 21:41:09 -0700 > Message-Id: <cover.1438057939.git.stillcompiling@xxxxxxxxx> > X-Mailer: git-send-email 2.4.6 > > The main goal of this series is to get rid of a needless typedef > in the rtl8712 wlan driver. > > In the course of fixing that, I found a bug that can (at least in theory) > lead to a overrun during a memcpy, as well as an identical struct with > a different name, which use the typedef. > Finally after cleaning up the typedef, change the name of the primary > variable that used the typedef from "SupportedRates" to "rates", to > conform to kernel coding style. > > Changes since V1: > Do not make other logic changes while Changing the name of SupportedRates > New patch 3/6 fixes a buggy comment that referred to the typedef > > Changes sinve V2: > broke former patch 4, which was too big for git-send-email into 3 parts > > Joshua Clayton (8): > staging: rtl8712: fix buggy size calculation > staging: rtl8712: simplify size calculation > staging: rtl8712: fix comment > staging: rtl8712: removed unused wrapper structs > staging: rtl8712: remove duplicate struct > staging: rtl8712: rename function > staging: rtl8712: remove typedefs > staging: rtl8712: change SupportedRates to rates > > drivers/staging/rtl8712/ieee80211.c | 22 ++++++------- > drivers/staging/rtl8712/rtl871x_cmd.c | 28 +++++----------- > drivers/staging/rtl8712/rtl871x_cmd.h | 18 ---------- > drivers/staging/rtl8712/rtl871x_event.h | 2 +- > drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 36 ++++++++++---------- > drivers/staging/rtl8712/rtl871x_mlme.c | 47 ++++++++++----------------- > drivers/staging/rtl8712/rtl871x_mlme.h | 2 +- > drivers/staging/rtl8712/rtl871x_mp_ioctl.c | 6 ++-- > drivers/staging/rtl8712/wlan_bssdef.h | 42 +++--------------------- > 9 files changed, 64 insertions(+), 139 deletions(-) > > -- > 2.4.6 > > _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
