diff options
| author | Pablo Barciela <[email protected]> | 2019-03-28 21:40:41 +0100 |
|---|---|---|
| committer | raveit65 <[email protected]> | 2022-07-19 23:16:06 +0200 |
| commit | fab059e8cdfb57112dac5b54492ff33fcbbdb62d (patch) | |
| tree | a112341de9e084371ace3f94eb47cd2030066aa9 | |
| parent | 485e3d8990ca9e4f4c39587db01d0facf954830a (diff) | |
| download | caja-fab059e8cdfb57112dac5b54492ff33fcbbdb62d.tar.bz2 caja-fab059e8cdfb57112dac5b54492ff33fcbbdb62d.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;
^~~~~~~~~~~~~~
| -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 968cf5e6..6de6fdfe 100644 --- a/libcaja-private/caja-file-operations.c +++ b/libcaja-private/caja-file-operations.c @@ -2162,7 +2162,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 { @@ -5445,7 +5445,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, |