Re: staging: rtl8188eu:Remove rtw_zmalloc(), wrapper for kzalloc()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jul 7, 2014 at 6:16 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> Hello navin patidar,
>
> The patch fadbe0cd5292: "staging: rtl8188eu:Remove rtw_zmalloc(),
> wrapper for kzalloc()" from Jun 22, 2014, leads to the following
> static checker warning:
>
>         drivers/staging/rtl8188eu/core/rtw_cmd.c:1234 rtw_setassocsta_cmd()
>         warn: struct type mismatch 'set_stakey_rsp vs set_assocsta_rsp'
>
>         drivers/staging/rtl8188eu/core/rtw_cmd.c:1234 rtw_setassocsta_cmd()
>         error: not allocating enough data 8 vs 4
>
> drivers/staging/rtl8188eu/core/rtw_cmd.c
>   1216          struct set_stakey_rsp *psetassocsta_rsp = NULL;
>                        ^^^^^^^^^^^^^^
>   1217
>   1218          u8      res = _SUCCESS;
>   1219
>   1220
>   1221          ph2c = kzalloc(sizeof(struct cmd_obj), GFP_KERNEL);
>   1222          if (ph2c == NULL) {
>   1223                  res = _FAIL;
>   1224                  goto exit;
>   1225          }
>   1226
>   1227          psetassocsta_para = kzalloc(sizeof(struct set_assocsta_parm), GFP_KERNEL);
>                                                           ^^^^^^^^^^^^^^^^^
>   1228          if (psetassocsta_para == NULL) {
>   1229                  kfree(ph2c);
>   1230                  res = _FAIL;
>   1231                  goto exit;
>   1232          }
>   1233
>   1234          psetassocsta_rsp = kzalloc(sizeof(struct set_assocsta_rsp), GFP_KERNEL);
>   1235          if (psetassocsta_rsp == NULL) {
>   1236                  kfree(ph2c);
>   1237                  kfree(psetassocsta_para);
>   1238                  return _FAIL;
>   1239          }
>   1240
>   1241          init_h2fwcmd_w_parm_no_rsp(ph2c, psetassocsta_para, _SetAssocSta_CMD_);
>   1242          ph2c->rsp = (u8 *)psetassocsta_rsp;
>   1243          ph2c->rspsz = sizeof(struct set_assocsta_rsp);
>                                             ^^^^^^^^^^^^^^^^
> These things should all match but I don't know the code well enough to
> say what the intent was.
>
> The code was buggy before you touched it, but your cleanup fixed it so
> my static checker could parse it.

Hi Dan,

rtw_setassocsta_cmd() is a unused function.
I've submitted a patch  "staging: rtl8188eu: Remove unused function
rtw_setassocsta_cmd()"
 to remove it.

regards,
navin patidar
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux