Chaine de caractère corrompue par un malloc

Version imprimable

Bonsoir,

Après avoir déclaré une chaine de caractère (la réservation mémoire, la mise à zéro et l'affectation fonctionnent sans problème, je peux l'afficher sans soucis), je fais un malloc pour une seconde chaine, mais cela semble corrompre ma chaine initiale:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 char *param1; char *param2 param1 = (char*)malloc(1+nbadresse * sizeof(void*)); memset(param1, 0, 1+nbadresse*sizeof(void*)); //J'initialise la chaine ici //.... printf("0 -> %s\n", param1); //Pas de soucis param2 = (char*)malloc(nb+strlen(code)+1); printf("1 -> %s\n", param1); //Ma chaine finis par un caractère différent et est coupée...
D'où cela peut-il venir?
Comment corriger ce problème?

Merci d'avance

@+
Seeme

Citation:

Envoyé par seeme

Après avoir déclaré une chaine de caractère (la réservation mémoire, la mise à zéro et l'affectation fonctionnent sans problème, je peux l'afficher sans soucis), je fais un malloc pour une seconde chaine, mais cela semble corrompre ma chaine initiale:

1 - un malloc() ne peut pas corrompre une chaine de caractères (ou alors il y a eu un comportement indéfini avat son appel).
2 - ton code est incomplet.
3 - ceci fonctionne sans problèmes :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 #include <stdio.h> #include <stdlib.h> #include <string.h> int main (void) { int nbadresse = 30; int nbNOP = 4; char shellcode[] = "hello world"; char *param1; char *param2; param1 = malloc (1 + nbadresse * sizeof (void *)); memset (param1, 0, 1 + nbadresse * sizeof (void *)); strcpy (param1, "J'initialise la chaine ici"); printf ("0 -> %s\n", param1); //Pas de soucis param2 = malloc (nbNOP + strlen (shellcode) + 1); printf ("1 -> %s\n", param1); return 0; }
Code:

1 2 3 4 5 6 0 -> J'initialise la chaine ici 1 -> J'initialise la chaine ici Process returned 0 (0x0) execution time : 0.021 s Press any key to continue.
Mais comme j'ai du inventer ce qui manque; j'ai peut être pas commis tes erreurs...

ceci est très louche :

Code:

param1 = malloc (1 + nbadresse * sizeof (void *));

Voilà le code un peu plus complet (je ne peut pas mettre la totallité du code ici)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
 
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <string.h>
#include <sys/types.h>
#include <sys/wait.h>
 
int main(int argc, char **argv) {
 
  int nb; // nombre
  char *param1; // param1 contient n fois l'adresse de retour
  char *param2;
 
  int adresse = 0;
  int nbadresse = 0;
  int i;
 
  char *prog = NULL; // nom du programme
 
  if (argc != 5) {
 
    printf("utilisation : ...");
    exit(1);
  }
 
  prog = argv[1];
 
 
  adresse = (int)strtol(argv[2],NULL,16);
  nbadresse = atoi(argv[3]);
  nb = atoi(argv[4]);
 
 
 
  // alloue param1, de taille 1+(nbadresse*taille d'une adresse)
  // met à zero tous les octets de param1 (cf. man memset)
 
 
  param1 = (char*)malloc(1+nbadresse * sizeof(void*));
  memset(param1, 0, 1+nbadresse*sizeof(void*));
 
  // rempli param1 avec nbadresse fois adresse
  /*
  for(i=0;i<nbadresse;i++) {
  	param1[i] = adresse;
  }
  */
 
	i = 0;
       int nb;
 for(i=0; i<(8*nbadresse); i+=8){
  for(nb = 0; nb < 8; nb+=2){
    param1[i+nb] = argv[2][9-nb-1];
    param1[i+nb+1] = argv[2][9-nb];
 } 
}
 
printf("0 -> %s\n", param1);
printf("1 -> %s\n", param1);
 
  // alloue param2, de taille 1+nb+taille du code
  // met à zero tous les octets de param2 (cf. man memset)
  /*
  param2 = malloc(1+nb+sizeof(code));
  memset(param2, 0, 1+nb+sizeof(code));
  */
 
  param2 = (char*)malloc(nb+strlen(code)+1);
printf("2 -> %s\n", param1);
  memset(param2, 0, 1+nb+strlen(code+1));
 
 
 
  for(i=0;i<nb;i++) {
  	param2[i] = 'a';
  }
 
  memcpy(param2 + nb, code, strlen(code));
 
 
printf("3 -> %s\n", param1);
 
  int  end = 0;
  int try = 1;
 
  int pid;
  int status;
 
 //....Ici on fork
  exit(0);
}

Le coup du char* vient de la source du prof, à l'origine, il utilisais un for comme ceci ensuite:

Code:

1
2
3
4
int* ptr = (int*) param1;
  for(i=0;i<nbadresse;i++) {
  	ptr[i] = adresse;
  }

Et ça ne posait pas de problème

08/02/2009, 01h53
Emmanuel Delahaye
Citation:

Envoyé par seeme

Voilà le code un peu plus complet (je ne peut pas mettre la totallité du code ici)

Déjà, ça
Code:

1 2 3 4 5 6 7 8 prog = argv[1]; adresse = (int)strtol(argv[2],NULL,16); nbadresse = atoi(argv[3]); nbNOP = atoi(argv[4]);
C'est faux. Le nom du programme est en [0]

Tous tes arguments de la ligne de commande sont décalés...

Ensuite :
Code:

1 2 3 4 utilisation : ... Process returned 1 (0x1) execution time : 0.025 s Press any key to continue.
ça n'aide pas beaucoup pour passer des arguments corrects...
08/02/2009, 01h55
seeme

prog correspond bien à argv[1], il ne désigne pas le nom du programme courant (argv[0]), mais le nom d'un autre programme (utilisé par un fork plus loin dans le programme.
08/02/2009, 01h55
Emmanuel Delahaye

Citation:

Envoyé par seeme

prog correspond bien à argv[1], il ne désigne pas le nom du programme courant (argv[0]), mais le nom d'un autre programme (utilisé par un fork plus loin dans le programme.

OK. Donne un exemple de ligne de commande valide.
08/02/2009, 01h59
seeme

Code:

./monprog cible 0xbff00104 3 10000

Citation:

Envoyé par seeme

Code:

./monprog cible 0xbff00104 3 10000

OK. Après corrections mineures il reste ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
 
-------------- Build: Debug in hello ---------------
 
Compiling: main.c
Linking console executable: bin\Debug\hello.exe
C:\dev\hello\main.c: In function `main':
C:\dev\hello\main.c:69: error: `shellcode' undeclared (first use in this function)
C:\dev\hello\main.c:69: error: (Each undeclared identifier is reported only once
C:\dev\hello\main.c:69: error: for each function it appears in.)
Process terminated with status 1 (0 minutes, 0 seconds)
3 errors, 0 warnings

Ensuite, il a un tas de code commenté. Je le retire ?

Ce qui est commenté doit le rester normalement (le programme peut fonctionner d'une autre manière en fait).

Mon output:
Code:

1 2 3 4 5 6 seeme@mercury:~/Programation/Failles$ ./exploit concat 0xbff00104 3 10000 0 -> 0401f0bf0401f0bf0401f0bf 1 -> 0401f0bf0401f0bf0401f0bf 2 -> 0401f0bf0401f0bf0401I' 3 -> 0401f0bf0401f0bf0401I' Fils: 0401f0bf0401f0bf0401I'