diff options
| author | Pablo Barciela <[email protected]> | 2019-03-28 04:44:36 +0100 |
|---|---|---|
| committer | raveit65 <[email protected]> | 2022-07-19 23:15:50 +0200 |
| commit | 485e3d8990ca9e4f4c39587db01d0facf954830a (patch) | |
| tree | 4e81a01f8472e5ef63a8992f7aed700b245b297c | |
| parent | 2e9c7635145a2045794220e4d5a5e279531d17d8 (diff) | |
| download | caja-485e3d8990ca9e4f4c39587db01d0facf954830a.tar.bz2 caja-485e3d8990ca9e4f4c39587db01d0facf954830a.tar.xz | |
caja-open-with-dialog: make sure there is error before showing it
Fixes Clang static analyzer warning:
caja-open-with-dialog.c:252:100: warning: Access to field 'message' results in a dereference of a null pointer (loaded from variable 'error')
message = g_strdup_printf (_("Could not add application to the application database: %s"), error->message);
^~~~~~~~~~~~~~
| -rw-r--r-- | libcaja-private/caja-open-with-dialog.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/libcaja-private/caja-open-with-dialog.c b/libcaja-private/caja-open-with-dialog.c index 45996fdf..e42a8fbd 100644 --- a/libcaja-private/caja-open-with-dialog.c +++ b/libcaja-private/caja-open-with-dialog.c @@ -249,12 +249,15 @@ add_or_find_application (CajaOpenWithDialog *dialog) if (app == NULL) { - message = g_strdup_printf (_("Could not add application to the application database: %s"), error->message); + message = g_strdup_printf (_("Could not add application to the application database: %s"), error ? error->message : _("Unknown error")); eel_show_error_dialog (_("Could not add application"), message, GTK_WINDOW (dialog)); g_free (message); - g_error_free (error); + + if (error) + g_error_free (error); + return NULL; } |