On 31/08/2017 19:01, Thomas Stüfe wrote:
Hi Alan,thank you for your review! Updated webrev:http://cr.openjdk.java.net/~stuefe/webrevs/8186665-buffer-overflow-in-mincore/webrev.01/webrev/ <http://cr.openjdk.java.net/%7Estuefe/webrevs/8186665-buffer-overflow-in-mincore/webrev.01/webrev/>I fixed the indention and embellished the comments around the sentinel at the end of the buffer somewhat.
This looks okay to me. -Alan