Merge "effects: fix a theoretical memory leak"

am: 7f4beda2de

Change-Id: I27d87188283c0e31cb607f3de15f9f1170cb94fb
gugelfrei
George Burgess IV 5 years ago committed by android-build-merger
commit 139b5728e2

@ -1856,12 +1856,13 @@ status_t AudioFlinger::EffectHandle::command(uint32_t cmdCode,
}
// copy to local memory in case of client corruption b/32220769
param = (effect_param_t *)realloc(param, size);
if (param == NULL) {
auto *newParam = (effect_param_t *)realloc(param, size);
if (newParam == NULL) {
ALOGW("command(): out of memory");
status = NO_MEMORY;
break;
}
param = newParam;
memcpy(param, p, size);
int reply = 0;

Loading…
Cancel
Save