Dado que el parámetro destiny[]
es un array de punteros, estos punteros deben estar inicializados, apuntando a una zona de memoria donde haya espacio suficiente para copiar la correspondiente cadena.
La causa más probable del segfault es que esos punteros estén sin inicializar. Por otro lado, inicializarlos de forma general no es fácil, pues la cantidad de memoria que cada uno necesite dependerá del tamaño de la cadena a copiar.
Para estas situaciones puede ser más útil usar strdup()
en lugar de strcpy()
. Esta función recibe como parámetro una cadena que queremos duplicar. Mide su longitud, reserva memoria para una copia, hace la copia, y devuelve el puntero a la copia.
Usando esa función harías:
void cpyCommand(char *origin[], char *destiny[]) {
int i = 0;
while (origin[i] != NULL) {
destiny[i] = strdup(origin[i]);
i++;
}
destiny[i] = NULL;
}
Nota Ten en cuenta que, de forma implícita, strdup()
ha llamado a malloc()
y por tanto tienes memoria que has reservado y deberías liberar. Cuando el array de copias ya no sea necesario, deberías recorrerlo haciendo un free()
a cada uno de sus elementos.
Nota2 Si alguna de las cadenas en el array origin
es extraordinariamente larga (o por un error no tiene un terminador), strdup()
puede agotar la memoria disponible en el heap. Para prevenirte ante este tipo de problemas (o ataques, si las cadenas origin
son suministradas por un usuario), puedes usar strndup()
en vez de strdup()
, ya que ésta te permite especificar un número máximo de caracteres a copiar.
strcpy(destiny, origin)
lo haría, sin necesidad de bucle. De hecho, el bucle sería un error, ya que en cada iteraciónorigin[i]
odestiny[i]
serían caracteres y no punteros a caracter como esperastrcpy()
. Si es lo segundo, lo más probable es que los arrays no estén bien inicializados, mejor muestras el código de la parte que los crea y llama a esta función. – abulafia el 25 may. 19 a las 21:10char *args[MAX_LINE/2]; char *history[300][MAX_LINE/2]; int histSize = 0; get_command(inputBuffer, MAX_LINE, args, &background); cpyCommand(args,history[histSize]); histSize++;
La función get_command() tengo comprobado que funciona perfectamente y escribe lo que tiene que escribir en el array de origen – jose_rullva el 25 may. 19 a las 22:32history
es un array de punteros a string? O un array de arrays de punteros a string? Porque tiene dos dimensiones, pero a la vez un*
delante. Y por otro lado ¿dónde se reserva la memoria para cada uno de los strings que copias? Creo que de ahí puede venirte la violación de segmento ya que todos esos punteros enhistory
no apuntan a ninguna dirección válida hasta que no los inicialices con un malloc. En este caso sería mejor usarstrdup()
. – abulafia el 25 may. 19 a las 22:54