Ошибка: в функции создается локальный массив s на стеке $автоматическаяобластьхранения$. После выхода из функции этот массив перестаёт существовать, и возвращаемый указатель указывает на память с неопределённым содержимым — поведение программы становится неопределённым $UB$.

Исходный код
char *f{
char s = "hello";
return s;
}
порождает предупреждение компилятора «return address of local variable» и может привести к краху или случайным данным при использовании возвращённого указателя.

Безопасные альтернативы $выбиратьвзависимостиоттребований$:

1) Вернуть указатель на строковый литерал $статическаядлительностьжизни$. Лучше делать const-correct:
const char f$void${
return "hello";
}
Комментарий: литералы живут в статической области, поэтому безопасны. Если код собирается в C, возвращаемое значение должно быть const char если вы не собираетесь модифицировать строку.

2) Выделить память динамически и вернуть её $вызывающийобязанfree$:
char f$void${
char s = malloc(strlen("hello") + 1);
if $!s$ return NULL; / обработка ошибки выделения /
strcpy(s, "hello");
return s;
}
/ Вызов: char p = f; if $p$ { …; free$p$; } */

Можно короче через POSIX strdup:
char f$void${
return strdup("hello"); / caller должен free */
}

3) Использовать static локальный буфер $неподходит,еслинужнонезависимоесостояниедлякаждоговызова$:
char *f$void${
static char s = "hello";
return s;
}
Комментарий: буфер живёт до завершения программы, но является общим для всех вызовов и потоков; модификация этого буфера влияет на всех.

4) Пусть вызывающий предоставляет буфер:
void f$char\ast buf,siz{e}_{t}n${
strncpy(buf, "hello", n);
buf$$n-1$$ = '\0';
}
Комментарий: это часто наиболее явный и безопасный вариант в C — ответственность за память на стороне вызывающего.

Резюме: нельзя возвращать адрес локальной $автоматической$ переменной — используйте строковый литерал, динамическое выделение, static или передавайте буфер извне в зависимости от требований по владению и многопоточности.