Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 412c80fc authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Clamp AIDL stream latency to [1, 3000] ms" into main am: 577aa365 am:...

Merge "Clamp AIDL stream latency to [1, 3000] ms" into main am: 577aa365 am: d2094760 am: e6affce8 am: 6538d144 am: 1e996265

Original change: https://android-review.googlesource.com/c/platform/frameworks/av/+/2719514



Change-Id: I44c73f3a36ec60d267fa1a7bf3583fb2c9843779
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents a2cfe84c 1e996265
Loading
Loading
Loading
Loading
+5 −1
Original line number Diff line number Diff line
@@ -267,7 +267,11 @@ status_t StreamHalAidl::getLatency(uint32_t *latency) {
    if (status_t status = updateCountersIfNeeded(&reply); status != OK) {
        return status;
    }
    *latency = std::max<int32_t>(0, reply.latencyMs);

    *latency = std::clamp(std::max<int32_t>(0, reply.latencyMs), 1, 3000);
    ALOGW_IF(reply.latencyMs != static_cast<int32_t>(*latency),
             "Suspicious latency value reported by HAL: %d, clamped to %u", reply.latencyMs,
             *latency);
    return OK;
}