This reverts commit
824ffcaf3a1bd55a111ed67cf5b0cb3cd18b405f.
The check is unnecessary because user_cmd.chanlist_len has already been
checked against the maximum supported by the subdevice.
Thanks to Dan Carpenter for pointing this out.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
async->cmd = user_cmd;
async->cmd.data = NULL;
/* load channel/gain list */
async->cmd = user_cmd;
async->cmd.data = NULL;
/* load channel/gain list */
- if (async->cmd.chanlist_len <= ULONG_MAX / sizeof(int))
- async->cmd.chanlist =
- kmalloc(async->cmd.chanlist_len * sizeof(int),
- GFP_KERNEL);
- else
- async->cmd.chanlist = NULL;
+ async->cmd.chanlist =
+ kmalloc(async->cmd.chanlist_len * sizeof(int), GFP_KERNEL);
if (!async->cmd.chanlist) {
DPRINTK("allocation failed\n");
ret = -ENOMEM;
if (!async->cmd.chanlist) {
DPRINTK("allocation failed\n");
ret = -ENOMEM;
/* load channel/gain list */
if (user_cmd.chanlist) {
/* load channel/gain list */
if (user_cmd.chanlist) {
- if (user_cmd.chanlist_len <= ULONG_MAX / sizeof(int))
- chanlist =
- kmalloc(user_cmd.chanlist_len * sizeof(int),
- GFP_KERNEL);
+ chanlist =
+ kmalloc(user_cmd.chanlist_len * sizeof(int),
+ GFP_KERNEL);
if (!chanlist) {
DPRINTK("allocation failed\n");
ret = -ENOMEM;
if (!chanlist) {
DPRINTK("allocation failed\n");
ret = -ENOMEM;