.. | .. |
---|
278 | 278 | |
---|
279 | 279 | tmp_len = (count > sizeof(tmp) - 1 ? sizeof(tmp) - 1 : count); |
---|
280 | 280 | |
---|
281 | | - if (!buffer || copy_from_user(tmp, buffer, tmp_len)) |
---|
282 | | - return count; |
---|
| 281 | + if (copy_from_user(tmp, buffer, tmp_len)) |
---|
| 282 | + return -EFAULT; |
---|
283 | 283 | |
---|
284 | 284 | tmp[tmp_len] = '\0'; |
---|
285 | 285 | |
---|
.. | .. |
---|
287 | 287 | num = sscanf(tmp, "%x %x %x", &addr, &val, &len); |
---|
288 | 288 | |
---|
289 | 289 | if (num != 3) |
---|
290 | | - return count; |
---|
| 290 | + return -EINVAL; |
---|
291 | 291 | |
---|
292 | 292 | switch (len) { |
---|
293 | 293 | case 1: |
---|
.. | .. |
---|
375 | 375 | |
---|
376 | 376 | tmp_len = (count > sizeof(tmp) - 1 ? sizeof(tmp) - 1 : count); |
---|
377 | 377 | |
---|
378 | | - if (!buffer || copy_from_user(tmp, buffer, tmp_len)) |
---|
379 | | - return count; |
---|
| 378 | + if (copy_from_user(tmp, buffer, tmp_len)) |
---|
| 379 | + return -EFAULT; |
---|
380 | 380 | |
---|
381 | 381 | tmp[tmp_len] = '\0'; |
---|
382 | 382 | |
---|
.. | .. |
---|
386 | 386 | if (num != 4) { |
---|
387 | 387 | rtl_dbg(rtlpriv, COMP_ERR, DBG_DMESG, |
---|
388 | 388 | "Format is <path> <addr> <mask> <data>\n"); |
---|
389 | | - return count; |
---|
| 389 | + return -EINVAL; |
---|
390 | 390 | } |
---|
391 | 391 | |
---|
392 | 392 | rtl_set_rfreg(hw, path, addr, bitmask, data); |
---|