test/wpackettest.c:79: error: INTEGER_OVERFLOW_L2
  ([0, +oo] - [0, 4]):unsigned64 by call to `WPACKET_finish`.
Showing all 10 steps of the trace


test/wpackettest.c:55:17: Call
53.               || !TEST_true(WPACKET_put_bytes_u8(&pkt, 0xff))
54.                   /* Closing a top level WPACKET should fail */
55.               || !TEST_false(WPACKET_close(&pkt))
                      ^
56.                   /* Finishing a top level WPACKET should succeed */
57.               || !TEST_true(WPACKET_finish(&pkt))

ssl/packet.c:243:1: Parameter `pkt->subs->lenbytes`
  241.   }
  242.   
  243. > int WPACKET_close(WPACKET *pkt)
  244.   {
  245.       /*

test/wpackettest.c:79:17: Call
77.       if (!TEST_true(WPACKET_init_len(&pkt, buf, 4))
78.               || !TEST_true(WPACKET_put_bytes_u8(&pkt, 0xff))
79.               || !TEST_true(WPACKET_finish(&pkt))
                      ^
80.               || !TEST_true(WPACKET_get_total_written(&pkt, &written))
81.               || !TEST_mem_eq(buf->data, written, simple3, sizeof(simple3)))

ssl/packet.c:255:1: Parameter `pkt->curr`
  253.   }
  254.   
  255. > int WPACKET_finish(WPACKET *pkt)
  256.   {
  257.       int ret;

ssl/packet.c:266:11: Call
  264.           return 0;
  265.   
  266.       ret = wpacket_intern_close(pkt, pkt->subs, 1);
                   ^
  267.       if (ret) {
  268.           OPENSSL_free(pkt->subs);

ssl/packet.c:188:1: <LHS trace>
186.    * (i.e. it fills in all the lengths), but doesn't actually close anything.
187.    */
188. > static int wpacket_intern_close(WPACKET *pkt, WPACKET_SUB *sub, int doclose)
189.   {
190.       size_t packlen = pkt->written - sub->pwritten;

ssl/packet.c:188:1: Parameter `pkt->curr`
    186.    * (i.e. it fills in all the lengths), but doesn't actually close anything.
    187.    */
    188. > static int wpacket_intern_close(WPACKET *pkt, WPACKET_SUB *sub, int doclose)
    189.   {
    190.       size_t packlen = pkt->written - sub->pwritten;

ssl/packet.c:188:1: <RHS trace>
186.    * (i.e. it fills in all the lengths), but doesn't actually close anything.
187.    */
188. > static int wpacket_intern_close(WPACKET *pkt, WPACKET_SUB *sub, int doclose)
189.   {
190.       size_t packlen = pkt->written - sub->pwritten;

ssl/packet.c:188:1: Parameter `sub->lenbytes`
    186.    * (i.e. it fills in all the lengths), but doesn't actually close anything.
    187.    */
    188. > static int wpacket_intern_close(WPACKET *pkt, WPACKET_SUB *sub, int doclose)
    189.   {
    190.       size_t packlen = pkt->written - sub->pwritten;

ssl/packet.c:203:13: Binary operation: ([0, +oo] - [0, 4]):unsigned64 by call to `WPACKET_finish` 
    201.   
    202.           /* Deallocate any bytes allocated for the length of the WPACKET */
    203.           if ((pkt->curr - sub->lenbytes) == sub->packet_len) {
                       ^
    204.               pkt->written -= sub->lenbytes;
    205.               pkt->curr -= sub->lenbytes;

static int wpacket_intern_close(WPACKET *pkt, WPACKET_SUB *sub, int doclose)
{
    size_t packlen = pkt->written - sub->pwritten;
    if (packlen == 0
            && (sub->flags & WPACKET_FLAGS_NON_ZERO_LENGTH) != 0)
        return 0;
    if (packlen == 0
            && sub->flags & WPACKET_FLAGS_ABANDON_ON_ZERO_LENGTH) {
        if (!doclose)
            return 0;
        if ((pkt->curr - sub->lenbytes) == sub->packet_len) {
            pkt->written -= sub->lenbytes;
            pkt->curr -= sub->lenbytes;
        }
        sub->packet_len = 0;
        sub->lenbytes = 0;
    }
    if (sub->lenbytes > 0
                && !put_value(&GETBUF(pkt)[sub->packet_len], packlen,
                              sub->lenbytes))
            return 0;
    if (doclose) {
        pkt->subs = sub->parent;
        OPENSSL_free(sub);
    }
    return 1;
}
['static int test_WPACKET_init(void)\n{\n    WPACKET pkt;\n    int i;\n    size_t written;\n    unsigned char sbuf[3];\n    if (!TEST_true(WPACKET_init(&pkt, buf))\n            || !TEST_true(WPACKET_put_bytes_u8(&pkt, 0xff))\n            || !TEST_false(WPACKET_close(&pkt))\n            || !TEST_true(WPACKET_finish(&pkt))\n            || !TEST_false(WPACKET_close(&pkt))\n            || !TEST_false(WPACKET_finish(&pkt))\n            || !TEST_true(WPACKET_get_total_written(&pkt, &written))\n            || !TEST_mem_eq(buf->data, written, simple1, sizeof(simple1)))\n        return cleanup(&pkt);\n    if (!TEST_true(WPACKET_init_len(&pkt, buf, 1))\n            || !TEST_true(WPACKET_put_bytes_u8(&pkt, 0xff))\n            || !TEST_true(WPACKET_finish(&pkt))\n            || !TEST_true(WPACKET_get_total_written(&pkt, &written))\n            || !TEST_mem_eq(buf->data, written, simple2, sizeof(simple2)))\n        return cleanup(&pkt);\n    if (!TEST_true(WPACKET_init_len(&pkt, buf, 4))\n            || !TEST_true(WPACKET_put_bytes_u8(&pkt, 0xff))\n            || !TEST_true(WPACKET_finish(&pkt))\n            || !TEST_true(WPACKET_get_total_written(&pkt, &written))\n            || !TEST_mem_eq(buf->data, written, simple3, sizeof(simple3)))\n        return cleanup(&pkt);\n    if (!TEST_true(WPACKET_init_len(&pkt, buf, 1)))\n        return cleanup(&pkt);\n    for (i = 1; i < 257; i++) {\n        if (!TEST_int_eq(WPACKET_put_bytes_u8(&pkt, 0xff), i < 256))\n            return cleanup(&pkt);\n    }\n    if (!TEST_true(WPACKET_finish(&pkt)))\n        return cleanup(&pkt);\n    if (!TEST_true(WPACKET_init_static_len(&pkt, sbuf, sizeof(sbuf), 0))\n            || !TEST_true(WPACKET_put_bytes_u24(&pkt, 0xffffff))\n            || !TEST_false(WPACKET_put_bytes_u8(&pkt, 0xff))\n            || !TEST_true(WPACKET_finish(&pkt))\n            || !TEST_true(WPACKET_get_total_written(&pkt, &written))\n            || !TEST_mem_eq(sbuf, written, fixed, sizeof(sbuf))\n            || !TEST_true(WPACKET_init_static_len(&pkt, sbuf, sizeof(sbuf), 1))\n            || !TEST_true(WPACKET_put_bytes_u16(&pkt, 0xfeff))\n            || !TEST_false(WPACKET_put_bytes_u8(&pkt, 0xff))\n            || !TEST_true(WPACKET_finish(&pkt))\n            || !TEST_true(WPACKET_get_total_written(&pkt, &written))\n            || !TEST_mem_eq(sbuf, written, alloc, sizeof(alloc)))\n        return cleanup(&pkt);\n    return 1;\n}', 'int WPACKET_close(WPACKET *pkt)\n{\n    if (pkt->subs == NULL || pkt->subs->parent == NULL)\n        return 0;\n    return wpacket_intern_close(pkt, pkt->subs, 1);\n}', 'int WPACKET_finish(WPACKET *pkt)\n{\n    int ret;\n    if (pkt->subs == NULL || pkt->subs->parent != NULL)\n        return 0;\n    ret = wpacket_intern_close(pkt, pkt->subs, 1);\n    if (ret) {\n        OPENSSL_free(pkt->subs);\n        pkt->subs = NULL;\n    }\n    return ret;\n}', 'static int wpacket_intern_close(WPACKET *pkt, WPACKET_SUB *sub, int doclose)\n{\n    size_t packlen = pkt->written - sub->pwritten;\n    if (packlen == 0\n            && (sub->flags & WPACKET_FLAGS_NON_ZERO_LENGTH) != 0)\n        return 0;\n    if (packlen == 0\n            && sub->flags & WPACKET_FLAGS_ABANDON_ON_ZERO_LENGTH) {\n        if (!doclose)\n            return 0;\n        if ((pkt->curr - sub->lenbytes) == sub->packet_len) {\n            pkt->written -= sub->lenbytes;\n            pkt->curr -= sub->lenbytes;\n        }\n        sub->packet_len = 0;\n        sub->lenbytes = 0;\n    }\n    if (sub->lenbytes > 0\n                && !put_value(&GETBUF(pkt)[sub->packet_len], packlen,\n                              sub->lenbytes))\n            return 0;\n    if (doclose) {\n        pkt->subs = sub->parent;\n        OPENSSL_free(sub);\n    }\n    return 1;\n}']