Allocazione dinamica di una stringa

Ciao, devo fare un metodo che data una stringa mi restituisce un nuovo puntatore ad una stringa con tutti i caratteri maiuscoli.
Ho scritto questo, ma mi da una marea di errori e non capisco dove sbaglio.

#include <stdio.h>
#include <mm_malloc.h>

char*  toUpperCase( *char s );
char*  toUpperCase( *char s ){
    //alloco la memoria per la nuova stringa
    char* p= malloc( strlen(s) +1 );
    //controllo se l'allocazione è andata a buon fine
    if ( p!= NULL ){
        //assegno a p i caratteri di s
        strcpy( p , s );
        int i;
        for(i=0;i< strlen(p); i++){
            // ogni carattere di p lo faccio diventare maiuscolo
            toupper(p[i]);
        }

    }
    return p;
}

int main(){
    printf("%s\n", toUpperCase("pippo") );
    return 0;
}

error: use of undeclared identifier 's' (perchè? grazie in anticipo)