Hi Staffan, thanks for having a look. On 10/21/2016 11:28 AM, Staffan Larsen wrote:
Can you change:185 if (total_length < 11) { to 185 if (total_length < sizeof(pktHeader) { ?
Yes, I'll then propose changing all 11 to sizeof(pktHeader). Full: http://rehn-ws.se.oracle.com/cr_mirror/8164501/01/webrev/ Inc: http://rehn-ws.se.oracle.com/cr_mirror/8164501/01_inc/webrev/ /Robbin
On 21 Oct 2016, at 09:41, Robbin Ehn <[email protected]> wrote: Hi all, please review! This patch makes sure pktHeader is not used uninitialized and that total_length is at least 11. The bug is confidential, but subject and patch says alot. Bug: https://bugs.openjdk.java.net/browse/JDK-8164501 Webrev: http://cr.openjdk.java.net/~rehn/8164501/webrev/ Thanks! /Robbin
