Итак, у меня есть два (надеюсь, быстрых) вопроса. Я думаю, что умею использовать malloc для экономии места для данных, но realloc вызывает проблемы. В приведенном ниже коде у меня есть массив из 8 символьных указателей, который - если он заполнится - я пытаюсь расширить до еще 8 символьных указателей (а затем еще 8 и т. Д. ). Realloc делает это в первый раз (т.е. расширяет массив один раз), но после этого я получаю следующую ошибку:
*** glibc detected *** ./a.out: realloc(): invalid next size:
Насколько я могу судить, ничего не меняется. Почему realloc будет работать с массивом из 8, но не с массивом из 16?
И второй вопрос касается утечек памяти. Я все еще не уверен, что мне нужно освободить в программе. Другие посоветовали мне освободить inputcpy. Это все здесь? Кроме того, на каком этапе программы я хочу его освободить?
#define DEBUG 1
#include <string.h>
#include <stdlib.h>
#include <stdio.h>
char ** getArgs( char *input, char **args, int ct);
char ** args;
int main(int argc, char* argv[]) {
char input[]="echo arg1 arg2 arg3 arg4 arg5 arg6 arg7 arg8 arg9 arg10 arg11 arg12 arg13";
char inputcpy[strlen(input)];
strcpy(inputcpy, input);
char * prog=strtok(input, " ");
/*Saving space for an array of 8 strings*/
args=( char **) calloc(8, sizeof( char *));
getArgs(inputcpy, args, 1);
if(DEBUG) {
printf("arg address after: %p\n", args);
}
int q;
int pid=fork();
if (pid==0) {
execvp(prog, args);
return 0;
}
else {
int status=0;
wait(&status);
}
}
char ** getArgs( char *input, char **args, int ct) {
int adj=(ct-1)*8;//if we recurse, this ensures correct indexes are used
char *inputcpy=malloc(strlen(input));
strcpy(inputcpy, input);
/*Initialize indexes/Prepare for copying*/
int i;
if(ct==1) {
i=1;
args[0]=" "; //quick hack to ensure all args are used by exec()
}
else
i=0;
/**Actually do the copying now**/
char *temp=strtok(NULL, " ");
args[adj+i++]=temp;
while (temp != NULL && i<8) {
temp=strtok(NULL, " ");
args[adj+i++]=temp;
}
/*If there are more args than we have room for*/
if(i>=8){
/*Increase the array to store 8 more strings*/
args= (char **) realloc(args, sizeof(args)+8*sizeof(char *) );
getArgs(inputcpy, args, (++ct) );
}
return NULL;
}
new
что-то,delete
это. Если вы что-тоmalloc
,free
это.sizeof
дает вам размерargs
(указатель), а не то, на что он указывает, что выглядит вашей ошибкой. - person Kieren Johnstone   schedule 26.02.2012char *inputcpy=malloc(strlen(input));
должно бытьchar *inputcpy=malloc(strlen(input) + 1);
, так как вы должны учитывать нулевой символ. - person Ed Heal   schedule 26.02.2012