diff options
author | Pablo Barciela <[email protected]> | 2019-03-28 21:40:41 +0100 |
---|---|---|
committer | monsta <[email protected]> | 2019-07-23 16:11:51 +0300 |
commit | 374757b8ff53c4e1a19827ccd04c1ac026e1e69e (patch) | |
tree | 4ffd877074e82bca87954720c43710c008f00e71 /libcaja-private | |
parent | 32b41e72a508636947fc3c107caf504fe74da224 (diff) | |
download | caja-374757b8ff53c4e1a19827ccd04c1ac026e1e69e.tar.bz2 caja-374757b8ff53c4e1a19827ccd04c1ac026e1e69e.tar.xz |
caja-file-operations: make sure 'error' isn't NULL
Fixes Clang static analyzer warnings:
caja-file-operations.c:2165:7: warning: Access to field 'code' results in a dereference of a null pointer (loaded from variable 'error')
if (error->code != G_IO_ERROR_FAILED_HANDLED) {
^~~~~~~~~~~
caja-file-operations.c:5448:14: warning: Access to field 'message' results in a dereference of a null pointer (loaded from variable 'error')
details = error->message;
^~~~~~~~~~~~~~
Diffstat (limited to 'libcaja-private')
-rw-r--r-- | libcaja-private/caja-file-operations.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libcaja-private/caja-file-operations.c b/libcaja-private/caja-file-operations.c index a12d730c..8ddc2ddf 100644 --- a/libcaja-private/caja-file-operations.c +++ b/libcaja-private/caja-file-operations.c @@ -2154,7 +2154,7 @@ unmount_mount_callback (GObject *source_object, } if (! unmounted) { - if (error->code != G_IO_ERROR_FAILED_HANDLED) { + if (error && error->code != G_IO_ERROR_FAILED_HANDLED) { if (data->eject) { primary = f (_("Unable to eject %V"), source_object); } else { @@ -5437,7 +5437,7 @@ link_file (CopyMoveJob *job, details = NULL; } else { secondary = f (_("There was an error creating the symlink in %F."), dest_dir); - details = error->message; + details = error ? error->message : NULL; } response = run_warning (common, |