Of course it would just end up being
PORT_Memcpy2(cx->u.buffer, sigLen, sig->data, sigLen); PORT_Memcpy2(cx->u.buffer, sigLen, sig->data, sigLen);I also wonder if a linter could notice that the dest buffer size passed isn’t the actual size of the buffer. (That leads the the next problem in the code, if you look at the definition of that buffer, so that’s good.)