Salut.
Je suis pas très calé en allocation de mémoire, mais je vais essayer de t'aider.
Je ne suis pas sur que sizeof s2 te donne la longueur de la chaine.
Tout d'abord, pour debugger, fait un printf de "sizeof s2" voir si cela correspond bien à la longueur de s2. Si oui, ne manquerai t'il pas une largeur de type ?
tu peux peut être remplacer sizeof s2 par strlen(s2).
Bonne chance Salutation !
Char Snipeur

Oula !
J'avais pas fait gaffe !
les s1[i] en dehors du for
à éviter.
vérifie sa valeur avant de l'utiliser, il est possible que le problème vienne de là Salutation !
Char Snipeur

Après test chez moi, en fait, il n'y a pas d'erreur.
Je pense donc que cela viens d'ailleur Salutation !
Char Snipeur

L'erreur de segmentation était à mon avis plutôt dûe au fait qu'il fallait écrire :

char *dst = (char *) malloc(sizeof(char)*strlen(src));

#include <stdlib.h>
#include <string.h>
#include <stdio.h>

char*
my_strcpy (char* dest, const char* src) {
  int i;
  dest = (char *) malloc(sizeof(char)*strlen(src));
  for (i = 0; src[i]!='\0'; ++i) dest[i] = src[i];
  dest[i] = '\0';
  return dest;
}

int main(){
  char *src = "tapir",dst;
  my_strcpy(dst,src);
  printf("dst = %s\n",dst);
  free(dst);
  getchar(); // pour les windowsiens
  return 0;
}

En même temps essayer de retourner des chaines de caractères en c ca devient risqué ^^
Le loup, solitaire et mystérieux.

Je ne vois pas pourquoi. En fait de manière générale on essaye juste de mettre le calloc/malloc dans le même horizon que le free (ce qui n'est pas le cas de l'exemple que j'ai donné) et c'est pour ça que la version de tafiscobar est plus propre. Il faut juste avec sa version penser à allouer la chaine dest dans le main (et à la désallouer).