New issue
Advanced search Search tips
Starred by 3 users
Status: Fixed
Owner:
Closed: Dec 2016
Cc:



Sign in to add a comment
Samsung: Heap-overflow in "tlc_server" via LOAD_TUI_RESOURCE command
Project Member Reported by laginimaineb@google.com, Sep 29 2016 Back to list
As a part of the TrustZone framework available on Samsung devices, Samsung provides an Android daemon which enables communication with a selected set of trustlets running in the Secure World.

The daemon is spawned multiple time, once for each supported trustlet. It then registers an Android binder service with a name corresponding to the trustlet with which it may communicate.

Apart from the regular commands which enable communication with the trustlet itself, the daemon also provides a special command called LOAD_TUI_RESOURCE (command code 5) which enables loading a KNOX TUI (Trusted User Interface) resource.

The function handling this command has the following logic:

Parcel data = ...;
...
if (data.readInt32(&numDwords))
    goto error;
uint32_t* buffer = (uint32_t*)malloc(numDwords * 4);
if (!buffer)
    goto error;
for (uint32_t i=0; i<numDwords; i++) {
    if (data.readInt32(&dword))
        goto error;
    buffer[i] = dword;
}

Since the length field (numDwords) is not validated, it may be arbitrarily large. This means that the calculation “4*numDwords” may overflow on 32-bit builds of "tlc_server" (i.e., on 32-bit Android devices), resulting in a small value.

The loop would then attempt to copy whatever amount of values written to the parcel (controlled by the attacker) into the allocated buffer, resulting in a heap-overflow.

The tlc_server runs with the “system” UID and GID and has an SELinux context of “u:r:tlc_server:s0”.

This bug is subject to a 90 day disclosure deadline. If 90 days elapse
without a broadly available patch, then the bug report will automatically
become visible to the public.
 
Project Member Comment 1 by laginimaineb@google.com, Dec 29 2016
Labels: -Restrict-View-Commit Deadline-Exceeded
Deadline exceeded -- automatically derestricting.
Project Member Comment 2 by laginimaineb@google.com, Dec 31 2016
Labels: SVE-2016-7220
Status: Fixed
Verified fix on version G925VVRU4CPK2_G925VVZW4CPK2_VZW
Project Member Comment 3 by laginimaineb@google.com, Feb 3 2017
Summary: Samsung: Heap-overflow in "tlc_server" via LOAD_TUI_RESOURCE command (was: Android: Heap-overflow in "tlc_server" via LOAD_TUI_RESOURCE command)
Sign in to add a comment