Shouldn't there be a strict comparison, since strlen() may return null?