Bruno Raoult

This is perfect, thanks a lot @stevenaaus ! And also bravo for the patch (your message below ).

I am trying to set a dark background, but I have one remaining issue (another one, hyperlinks color, was already answered in another discussion ) : In "Game List" window, the active game line has a a blue text, which is difficult to read. Is there an option to change this color ? Thanks a lot. Version 4.24, Tcl/Tk 8.6.13.

I can see issues in CuAssertMemEquals_LineMsg: 1) If one of actual/expected is NULL, we will get a segfault: if ((expected == NULL && actual == NULL) || // both NULL (expected != NULL && actual != NULL && // both non NULL (memcmp(expected, actual, len)) == 0)) { return; } /* below, dereferenced expected/actual are used, but we may have * one NULL, and get segfault. */ 2) buf is filled before checking for overflow. It means we can have an overflow: For example: c += sprintf(&buf[c],"expected <\n");...

I can see issues in CuAssertMemEquals_LineMsg: 1) If one of actual/expected is NULL, we will get a segfault: if ((expected == NULL && actual == NULL) || (expected != NULL && actual != NULL && (memcmp(expected, actual, len)) == 0)) { return; } // after that, dereferenced expected/actual are used, wrong if NULL 2) buf is filled before checking for overflow. It means we can have an overflow: For example: for(i=0;i < len;i++) c += sprintf(&buf[c],"%s0x%02x",((i % 16) == 15 ) ? "\n" : (i > 0) ? "," :...