diff options
| author | Deepak Kodihalli <dkodihal@in.ibm.com> | 2018-01-29 08:32:53 -0600 |
|---|---|---|
| committer | Deepak Kodihalli <dkodihal@in.ibm.com> | 2018-01-29 08:36:20 -0600 |
| commit | b5de286c50d2f9204243e69b49222e8d0fe7204f (patch) | |
| tree | 6755c5b472f17f1438a99f6066d13d002acf5100 /sensorhandler.cpp | |
| parent | e73d15d5f5ea12ff94afb77c1e7f1867bd4c5acf (diff) | |
| download | phosphor-host-ipmid-b5de286c50d2f9204243e69b49222e8d0fe7204f.tar.gz phosphor-host-ipmid-b5de286c50d2f9204243e69b49222e8d0fe7204f.zip | |
Remove unwarranted call to free()
Do not free() the pointer allocated by sd_bus_message_read(), because
systemd will free that once the message is unreferenced. The free() in
question here was causing an ipmid core dump.
Resolves openbmc/openbmc#2846.
Change-Id: I118f6947e7292b68744d0034c72572e4dcb99f01
Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Diffstat (limited to 'sensorhandler.cpp')
| -rw-r--r-- | sensorhandler.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/sensorhandler.cpp b/sensorhandler.cpp index 9064ecc..c8505e7 100644 --- a/sensorhandler.cpp +++ b/sensorhandler.cpp @@ -573,7 +573,6 @@ ipmi_ret_t ipmi_sen_get_sensor_reading(ipmi_netfn_t netfn, ipmi_cmd_t cmd, resp->indication[0] = 0x1; } resp->indication[1] = 0; - free(assertion); break; case IPMI_SENSOR_TEMP: |
