Just making sure

I was pouring over some code recently and came across one expression that really made me shake my head:

buff[strlen(buff)] = 0;

I pretty sure the intent was

buff[strlen(buff) - 1] = 0;

Either that, or they really wanted to make sure the string was null-terminated.