diff options
author | david.chiu@tw.synaptics.com <david.chiu@tw.synaptics.com> | 2020-10-23 15:03:05 +0800 |
---|---|---|
committer | david.chiu@tw.synaptics.com <david.chiu@tw.synaptics.com> | 2020-10-23 15:03:05 +0800 |
commit | 931b61ffc31aece93517fec3a04dcefea60e5656 (patch) | |
tree | 3824f958fdbe071342db924a0884d6c8db2c90ec | |
parent | 9c8fb8cedfd7474c837b75e2daf000754429721c (diff) | |
download | rmi4utils-931b61ffc31aece93517fec3a04dcefea60e5656.tar.gz |
Initialize varibales as NULL to avoid segmentation fault even no allocation as below.
-rwxr-xr-x | rmi4update/rmi4update.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/rmi4update/rmi4update.cpp b/rmi4update/rmi4update.cpp index cbb4d30..0eb6c7f 100755 --- a/rmi4update/rmi4update.cpp +++ b/rmi4update/rmi4update.cpp @@ -405,6 +405,11 @@ int RMI4Update::ReadFlashConfig() free(data_temp); } + // Initialize as NULL here to avoid segmentation fault. + m_partitionConfig = NULL; + m_partitionCore = NULL; + m_partitionGuest = NULL; + /* parse the config length */ for (i = 2; i < m_blockSize * m_flashConfigLength; i = i + 8) { @@ -439,7 +444,7 @@ int RMI4Update::ReadFlashConfig() if (partition_temp) free(partition_temp); - + m_fwBlockCount = m_partitionCore ? m_partitionCore->partition_len : 0; m_configBlockCount = m_partitionConfig ? m_partitionConfig->partition_len : 0; m_guestBlockCount = m_partitionGuest ? m_partitionGuest->partition_len : 0; |