display: Fix null pointer dereference
Change-Id: I9737741d066944e004f3e3a408fcb6ccd10282a3
This commit is contained in:
committed by
Gerrit - the friendly Code Review server
parent
19349d7192
commit
3053e37f77
@@ -753,12 +753,16 @@ void DeviceImpl::DeviceClientContext::ParseGetSupportedDisplayRefreshRates(
|
|||||||
|
|
||||||
uint32_t *refresh_rates_data =
|
uint32_t *refresh_rates_data =
|
||||||
reinterpret_cast<uint32_t *>(malloc(sizeof(uint32_t) * refresh_rates.size()));
|
reinterpret_cast<uint32_t *>(malloc(sizeof(uint32_t) * refresh_rates.size()));
|
||||||
for (int i = 0; i < refresh_rates.size(); i++) {
|
if (refresh_rates_data) {
|
||||||
refresh_rates_data[i] = refresh_rates[i];
|
for (int i = 0; i < refresh_rates.size(); i++) {
|
||||||
|
refresh_rates_data[i] = refresh_rates[i];
|
||||||
|
}
|
||||||
|
output_params.setToExternal(reinterpret_cast<uint8_t *>(refresh_rates_data),
|
||||||
|
sizeof(uint32_t) * refresh_rates.size());
|
||||||
|
_hidl_cb(error, output_params, {});
|
||||||
|
} else {
|
||||||
|
_hidl_cb(-EINVAL, {}, {});
|
||||||
}
|
}
|
||||||
output_params.setToExternal(reinterpret_cast<uint8_t *>(refresh_rates_data),
|
|
||||||
sizeof(uint32_t) * refresh_rates.size());
|
|
||||||
_hidl_cb(error, output_params, {});
|
|
||||||
}
|
}
|
||||||
|
|
||||||
Return<void> DeviceImpl::perform(uint64_t client_handle, uint32_t op_code,
|
Return<void> DeviceImpl::perform(uint64_t client_handle, uint32_t op_code,
|
||||||
|
|||||||
Reference in New Issue
Block a user