[davenportw15]

Wouldn't be cleaner to implement is_valid like this:

  int is_valid(char* password) {
    return strcmp(password, "poop") == 0;
  }

[codygman]

Maybe, but if/else is more approachable especially if you haven't seen that pattern before.