colin/ModemManager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

base-modem-at: make abort_task_if_port_unusable() unref the GTask

Browse Source 
Aleksander seems to prefer it this way. Looks better.

Suggested-by: Aleksander Morgado <aleksandermj@chromium.org>
This commit is contained in:
Lubomir Rintel
2023-01-17 08:34:02 +01:00
committed by Aleksander Morgado
parent f0a77a2292
commit da55fa769e
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/mm-base-modem-at.c
View File

@@ -35,6 +35,7 @@ abort_task_if_port_unusable (MMBaseModem *self,
MM_CORE_ERROR,
MM_CORE_ERROR_NOT_FOUND,
"Cannot run sequence: port not given");
g_object_unref (task);
return FALSE;
}
@@ -44,6 +45,7 @@ abort_task_if_port_unusable (MMBaseModem *self,
MM_CORE_ERROR,
MM_CORE_ERROR_CONNECTED,
"Cannot run sequence: port is connected");
g_object_unref (task);
return FALSE;
}
@@ -60,6 +62,7 @@ abort_task_if_port_unusable (MMBaseModem *self,
"Cannot run sequence: '%s'",
error->message);
g_error_free (error);
g_object_unref (task);
return FALSE;
}
@@ -235,10 +238,8 @@ at_sequence_common (MMBaseModem *self,
AtSequenceContext *ctx;
/* Ensure that we have an open port */
if (!abort_task_if_port_unusable (self, port, task)) {
g_object_unref (task);
if (!abort_task_if_port_unusable (self, port, task))
return;
}
/* Setup context */
ctx = g_new0 (AtSequenceContext, 1);
@@ -540,10 +541,8 @@ at_command_common (MMBaseModem *self,
AtCommandContext *ctx;
/* Ensure that we have an open port */
if (!abort_task_if_port_unusable (self, port, task)) {
g_object_unref (task);
if (!abort_task_if_port_unusable (self, port, task))
return;
}
ctx = g_new0 (AtCommandContext, 1);
ctx->port = g_object_ref (port);