Skip to content

epson2: check for return value of e2_esc_cmd

Zdenek Dohnal requested to merge zdohnal/backends:esci_set_focus_retval into master

Hi,

I ran our internal coverity scanner (tool which aggregates reports from cppcheck, strict gcc and clang compilation, shellcheck) for several CVE fixes from 1.0.30 and found the issue mentioned above.

esci_set_focus_position, which is a macro for e2_esc_cmd, returns status value, but it is never checked.

The patch fixes the warning, compiles fine, but I don't have any epson2 supported device to check it on the real device. Though it doesn't have an impact on the functionality IMO.

Would you mind merging the fix?

Thank you in advance and have a nice day,

Zdenek

Merge request reports