From: Paul Clarke > Sent: 04 May 2017 16:07 ... > > + if ((c = strnchr(name, MODULE_NAME_LEN, ':')) != NULL) { > > Shouldn't this be MODULE_NAME_LEN + 1, since the ':' can come after a module name of length > MODULE_NAME_LEN? No, because MODULE_NAME_LEN includes the terminating '\0'. David