diff options
| author | Christopher Michael <cp.michael@samsung.com> | 2019-03-12 09:57:33 -0400 |
|---|---|---|
| committer | Stefan Schmidt <s.schmidt@samsung.com> | 2019-03-13 11:51:58 +0100 |
| commit | 86cf4e5d7be0cdbdb51149c9b8fe9ea6e3e3d694 (patch) | |
| tree | b5bab1d950ac27258cdb7f81077050167adcb3b3 | |
| parent | 02d47fd61a007a654a782e83517ea3531921316b (diff) | |
| download | efl-86cf4e5d7be0cdbdb51149c9b8fe9ea6e3e3d694.tar.gz | |
elm_config: Fix unchecked return value
Small patch to check the return value of ecore_file_cp. Coverity
reports this as an unchecked return value, so let's just add a simple
check here.
Fixes Coverity CID1399101
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8309
| -rw-r--r-- | src/lib/elementary/elm_config.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/lib/elementary/elm_config.c b/src/lib/elementary/elm_config.c index a6d943e356..a88d62503e 100644 --- a/src/lib/elementary/elm_config.c +++ b/src/lib/elementary/elm_config.c @@ -779,7 +779,8 @@ _elm_config_profile_derived_save(const char *profile, Elm_Config_Derived *derive eet_close(ef); if (ret) { - ecore_file_cp(buf, buf2); + if (!ecore_file_cp(buf, buf2)) + ERR("Error saving Elementary's derived configuration profile file"); ecore_file_unlink(buf); } else |