I stand by my comment. :-)  The caller passes the return value to strlen().  This code looks suspect.