diff options
author | Dan Williams <dcbw@redhat.com> | 2010-02-22 11:06:50 -0800 |
---|---|---|
committer | Dan Williams <dcbw@redhat.com> | 2010-02-22 11:06:50 -0800 |
commit | 10e5e6561f0d0cef5dfa52744fe3ca3abf062684 (patch) | |
tree | a9d3670174d93ed7c460929ec6d4ca226f16b1f5 /libqcdm/src/commands.c | |
parent | 0f9d4d2a1ac3414b25c71f736c5b1293e1595721 (diff) |
qcdm: handle more command error responses
Diffstat (limited to 'libqcdm/src/commands.c')
-rw-r--r-- | libqcdm/src/commands.c | 32 |
1 files changed, 31 insertions, 1 deletions
diff --git a/libqcdm/src/commands.c b/libqcdm/src/commands.c index a3052007..e757cef4 100644 --- a/libqcdm/src/commands.c +++ b/libqcdm/src/commands.c @@ -32,7 +32,37 @@ check_command (const char *buf, gsize len, guint8 cmd, gsize min_len, GError **e return FALSE; } - if (buf[0] != (guint8) cmd) { + switch (buf[0]) { + case DIAG_CMD_BAD_CMD: + g_set_error (error, QCDM_COMMAND_ERROR, QCDM_COMMAND_BAD_COMMAND, + "DM command %d unknown or unimplemented by the device", + cmd); + return FALSE; + case DIAG_CMD_BAD_PARM: + g_set_error (error, QCDM_COMMAND_ERROR, QCDM_COMMAND_BAD_PARAMETER, + "DM command %d contained invalid parameter", + cmd); + return FALSE; + case DIAG_CMD_BAD_LEN: + g_set_error (error, QCDM_COMMAND_ERROR, QCDM_COMMAND_BAD_LENGTH, + "DM command %d was the wrong size", + cmd); + return FALSE; + case DIAG_CMD_BAD_DEV: + g_set_error (error, QCDM_COMMAND_ERROR, QCDM_COMMAND_NOT_ACCEPTED, + "DM command %d was not accepted by the device", + cmd); + return FALSE; + case DIAG_CMD_BAD_MODE: + g_set_error (error, QCDM_COMMAND_ERROR, QCDM_COMMAND_BAD_MODE, + "DM command %d not allowed in the current device mode", + cmd); + return FALSE; + default: + break; + } + + if (buf[0] != cmd) { g_set_error (error, QCDM_COMMAND_ERROR, QCDM_COMMAND_UNEXPECTED, "Unexpected DM command response (expected %d, got %d)", cmd, buf[0]); |