tree 6cf12dd5a34ca8c007ed14cfb509138fbc1ef535
parent eb72ff058fa4ae84b5e2d41f9973009647cf2a74
author Neels Hofmeyr <neels@hofmeyr.de> 1536005148 +0200
committer Neels Hofmeyr <neels@hofmeyr.de> 1536008888 +0200

fix mgcp_verify_ci(): off-by-one in max len check

MGCP_CONN_ID_MAXLEN actually includes a terminating nul, so we need to compare
strlen() against MGCP_CONN_ID_MAXLEN-1.

Log the length if it is too long.

Add MDCX_TOO_LONG_CI test to mgcp_test.c, testing a conn id of 33 characters.
Before this patch, the test returns error code 515 meaning "not found", while
now it returns 510 meaning "invalid", showing the off-by-one. Same is
illustrated by the error log ("not found" before, "too long" now), but the
error log is not verified by mgcp_test.c.

Change-Id: I8d6cc96be252bb486e94f343a8c7cae641ff9429