It's hard to know what "correct" means, because the code is incomplete - I don't for instance, know what "updatenewstatus" is.
void newthings(char* s)
{
char buffer [80];
sprintf(test, "HELP: it is very cold outside %s", s); // A more defensive, less RAM-consuming approach may be needed here.
}