[SOLUCIONADO] error usando cadenas

[eke_ps]

hola , el ejercicio que trato de hacer es simple
insertar una sub cadena en otra, a partir de una posicion dada

me funciona bien(compila todo), pero me salta un error de debug en una ventanita que dice omitir o continuar, se ve el resultado, el cual esta bien pero la ventanita me bajaria puntos si me pasaria en alguno de mis examenes

les dejo el codigo y abajo el error

Código C:

#include <stdio.h>
#include <string.h>
 
void insertarSub(char cad1[], char cad2[], int pos);
int main()
{
    char micad1[] = "hola ";
    char micad2[] = "como";
    insertarSub(micad1, micad2,2);
    return 0;
}
 
void insertarSub(char cad1[], char cad2[], int pos)
{
    int i = pos;
    int j = 0;
    int lar = strlen(cad2);
    char aux[100];
    char aux0[100];
 
    for (i = 0; i < pos; i++)
    {
        aux0[j] = cad1[i];
        j++;
    }
    aux0[j] = '\0';
    j = 0;
    puts(aux0);
 
    for (i= pos; cad1[i]; i++)
    {
        aux[j] = cad1[i];
        j++;
    }
    aux[j] = '\0';
    strcat(aux0, cad2);
    puts(aux0);
    strcat(aux0, aux);
    puts(aux0);
 
    strcpy(cad1, "");
    
    strcpy(cad1, aux0);
    puts(cad1);
}
 

run-time check failure#2 - stack around the variable 'micad1' was corrupted



si comento el ultimo strcat funciona bien, el tema es con micad1 :/

[CalgaryCorpus]

No tienes espacio suficiente en el string de destino para copiar los datos que intentas copiar.

[yoel_monsalve]

En efecto, suponiendo que el código de insertarSub sea correcto (no le he revisado a fondo), el problema es que la declaración

Código C++:

char micad1[] = "hola ";
 

informa al compilador que reserve en micad1 el espacio justo para la cadena "hola". Y por cierto que, si mi memoria no me falla, esta declaración implica además que la cadena declarada es tipo "const char *", es decir, una cadena constante cuyo contenido no puede ser cambiado (!!)

Entonces, ¿cómo vas a insertar una cadena dentro de micad1, si esto implica alterarla? No puedes.

Debes modificar la declaración de micad1, pudiendo quedar más o menos de este modo:

Código C++:

char micad1[100];       /* espacio suficiente */
 
strcpy(micad1, "hola");
 

ahora cad1 no será una cadena constante, y además contendrá el espacio que necesita para lo que quieres hacer. Pruébalo y comenta .....

[CalgaryCorpus]

Lo indicado esta casi bien.

Es cierto que "hola" es un string constante y como tal no se puede modificar, pero al inicializar un arreglo de caracteres con un string constante, el cargador hace la copia para ti, desde la zona constante hacia la variable, gratis.

El string constante existe, y la variable inicializada también. Son 2 entes independientes, usualmente guardados en zonas diferentes de memoria.

Mini.ejemplo:

Código C++:

#include <iostream>
using namespace std;
 
int main()
{
   char h[] = "hola que tal";
 
   cout << "ANTES   : " << h << endl;
   h[0] = 'c';
 
   cout << "DESPUES : " << h << endl;
}
 

Verlo funcionando aqui: http://goo.gl/rWQAy8

[eke_ps]

muchas gracias a todos, lo pude arreglar de las maneras que me indicaron!