| .. | .. |
|---|
| 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); |
|---|