`s` may be null, and so the strlen may seg fault.
And that's what is wrong. This is the most unfriendly behavior towards the programmer.
If I program it to cause a null pointer seg fault, I expect a null pointer seg fault. If I program it to cause a twos complement overflow, I want a twos complement overflow.