summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Schmidt <s.schmidt@samsung.com>2020-04-17 14:34:21 +0200
committerStefan Schmidt <s.schmidt@samsung.com>2020-04-23 13:06:33 +0200
commit745270061883d9047b90b4bb53084729d3c60d35 (patch)
tree02743cc250087aaad5d7ea06fa71fad3b0e06a74
parent1c971fbe40a572e49fa2ad5c8f0745a69e0a4c63 (diff)
exactness: do not access potential NULL pointer
These to code lines should be in one block and not one exectued without the if. CID: 1422198 Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11726
-rw-r--r--src/bin/exactness/player.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/src/bin/exactness/player.c b/src/bin/exactness/player.c
index 2dcf3f5e59..a3d6b04880 100644
--- a/src/bin/exactness/player.c
+++ b/src/bin/exactness/player.c
@@ -1014,8 +1014,11 @@ _write_unit_file(void)
1014 Exactness_Unit *tmp = NULL; 1014 Exactness_Unit *tmp = NULL;
1015 1015
1016 EINA_SAFETY_ON_NULL_RETURN(_src_unit); 1016 EINA_SAFETY_ON_NULL_RETURN(_src_unit);
1017 if (_src_type == FTYPE_EXU) tmp = exactness_unit_file_read(_src_filename); 1017 if (_src_type == FTYPE_EXU)
1018 _dest_unit->actions = tmp->actions; 1018 {
1019 tmp = exactness_unit_file_read(_src_filename);
1020 _dest_unit->actions = tmp->actions;
1021 }
1019 exactness_unit_file_write(_dest_unit, _dest); 1022 exactness_unit_file_write(_dest_unit, _dest);
1020 } 1023 }
1021} 1024}