lkml.org 
[lkml]   [2017]   [May]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [Linux-ima-devel] [PATCH 0/7] IMA: new parser for ima_restore_measurement_list()
From
Date
On 5/18/2017 5:38 AM, Roberto Sassu wrote:
>
> There cannot be buffer overflow, because the length of each digest
> field is known.

Crypto Agile: pcr[4] total_digest_len[4]
digest1_len[4] digest1[digest1_len] ...

The way I read this, the digest length is supplied by the caller, which
is slightly different from "known". For example, if I supply a digest
length of 0xffffffff, a too trusting (buggy) parser could overflow the
buffer.

total_digest_len is similarly untrusted. The attacker can send invalid
values.


\
 
 \ /
  Last update: 2017-05-23 23:01    [W:0.176 / U:0.772 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site