Wouldn't it be better to make `sid` `const char *`?