I'm talking about this line, where atoll() is the more appropriate choice. unsigned long long number = atol(Str);
unsigned long long number = atol(Str);