diff options
author | Stephan Gerhold <stephan.gerhold@kernkonzept.com> | 2022-08-11 17:23:20 +0200 |
---|---|---|
committer | Stephan Gerhold <stephan.gerhold@kernkonzept.com> | 2022-08-18 15:35:35 +0200 |
commit | 2b83e274742039432de81f663c34383c238d0314 (patch) | |
tree | d5ee44d86a7030f8abb1c290964c994a6b3f73b9 | |
parent | 23c027a7a9b916aabc82e70d922b727aa70af89a (diff) |
broadband-modem-qmi: add missing error check for MHI
Now that peek_port_qmi_for_data_mhi() is a separate function it becomes
obvious that it looks different from all the others: There is no check
that a QMI port was actually found.
Add it similar to the one used for qmi_wwan.
-rw-r--r-- | src/mm-broadband-modem-qmi.c | 15 |
1 files changed, 14 insertions, 1 deletions
diff --git a/src/mm-broadband-modem-qmi.c b/src/mm-broadband-modem-qmi.c index 75c07dd8..9619acb6 100644 --- a/src/mm-broadband-modem-qmi.c +++ b/src/mm-broadband-modem-qmi.c @@ -267,7 +267,20 @@ peek_port_qmi_for_data_mhi (MMBroadbandModemQmi *self, QmiSioPort *out_sio_port, GError **error) { - return mm_broadband_modem_qmi_peek_port_qmi (self); + MMPortQmi *found = NULL; + + found = mm_broadband_modem_qmi_peek_port_qmi (self); + + if (!found) + g_set_error (error, + MM_CORE_ERROR, + MM_CORE_ERROR_NOT_FOUND, + "Couldn't find associated QMI port for 'net/%s'", + mm_port_get_device (data)); + else if (out_sio_port) + *out_sio_port = QMI_SIO_PORT_NONE; + + return found; } static MMPortQmi * |