change type for strlen to unsigned long