diff options
| author | James Dong <jdong@google.com> | 2012-03-13 19:20:55 -0700 |
|---|---|---|
| committer | James Dong <jdong@google.com> | 2012-03-13 19:23:23 -0700 |
| commit | 35250664455c9642cc14d3831b663880637a7d70 (patch) | |
| tree | 5235afea54b20fde679fe3f4b1ac8c7442a4376f /drm | |
| parent | aa4397f07c43bd83bc3100b749401dc3d15e7622 (diff) | |
| download | frameworks_av-35250664455c9642cc14d3831b663880637a7d70.zip frameworks_av-35250664455c9642cc14d3831b663880637a7d70.tar.gz frameworks_av-35250664455c9642cc14d3831b663880637a7d70.tar.bz2 | |
Fixed a buffer overflow bug in DrmPassthruPlugin
Change-Id: I8df2a90409c9266a094a1a0904a5ff76ec483d16
Diffstat (limited to 'drm')
| -rw-r--r-- | drm/libdrmframework/plugins/passthru/src/DrmPassthruPlugIn.cpp | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/drm/libdrmframework/plugins/passthru/src/DrmPassthruPlugIn.cpp b/drm/libdrmframework/plugins/passthru/src/DrmPassthruPlugIn.cpp index 0ffc0a7..77db32c 100644 --- a/drm/libdrmframework/plugins/passthru/src/DrmPassthruPlugIn.cpp +++ b/drm/libdrmframework/plugins/passthru/src/DrmPassthruPlugIn.cpp @@ -279,8 +279,14 @@ status_t DrmPassthruPlugIn::onDecrypt(int uniqueId, DecryptHandle* decryptHandle * memory has to be allocated by the caller. */ if (NULL != (*decBuffer) && 0 < (*decBuffer)->length) { - memcpy((*decBuffer)->data, encBuffer->data, encBuffer->length); - (*decBuffer)->length = encBuffer->length; + if ((*decBuffer)->length >= encBuffer->length) { + memcpy((*decBuffer)->data, encBuffer->data, encBuffer->length); + (*decBuffer)->length = encBuffer->length; + } else { + ALOGE("decBuffer size (%d) too small to hold %d bytes", + (*decBuffer)->length, encBuffer->length); + return DRM_ERROR_UNKNOWN; + } } return DRM_NO_ERROR; } |