On Sat, Apr 22, 2017 at 1:56 PM, Maksim Salau <maksim.sa...@gmail.com> wrote:
> Allocate buffer on HEAP instead of STACK for a local structure
> that is to be sent using usb_control_msg().
>
> Signed-off-by: Maksim Salau <maksim.sa...@gmail.com>
> ---
> drivers/net/can/usb/gs_usb.c | 17 ++++++++++++-----
> 1 file changed, 12 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/net/can/usb/gs_usb.c b/drivers/net/can/usb/gs_usb.c
> index a0dabd4..98f972a 100644
> --- a/drivers/net/can/usb/gs_usb.c
> +++ b/drivers/net/can/usb/gs_usb.c
> @@ -740,13 +740,18 @@ static const struct net_device_ops gs_usb_netdev_ops = {
> static int gs_usb_set_identify(struct net_device *netdev, bool do_identify)
> {
> struct gs_can *dev = netdev_priv(netdev);
> - struct gs_identify_mode imode;
> + struct gs_identify_mode *imode = NULL;
No need to assign imode to NULL here.
