Crash if iptr->name is NULL.  Given that you're using SafeStringLength() to test its length, rather than strlen(), you're allowing that this is a possible state of affairs.  It's also over-kill to call SafeStringLength() on ipbuf, which definitely isn't NULL.  So I think the conditional above should be: <code>
if (iptr->name && strlen(ipbuf) + strlen(iptr->name) < sizeof(ipbuf))
</code>