🚀 go-pugleaf

Comento tu codigo...

| void main()
| {
|         char *cad1, *cad2;
|         int i=0, limsup=0;
|         cad1=(char*)malloc(20*sizeof(char*));
|         cad2=(char*)malloc(20*sizeof(char*));

Al usar malloc, en vez de sizeof (char *) tendr�as que usar sizeof(char), ya que reservas
memoria para una cadena de caracteres, no de punteros.

Si trabajas en C++, es aconsejable ser�a usar el m�todoespec�fico de C++ para reservar
memoria (new):

|         cad1="";

Buf... Esta asignaci�n es tremenda. Inicialmente tenias la variable cad1 apuntando a la
zona de memoria que habias reservado, y asi est�s haciendo que apunte a otro sitio
diferente.

La forma correcta de inicializaci�n ser�a

memset (cad1, '\0', 20);

o si solamente quieres poner a '\0' el primer caracter

cad1[0] = '\0';
 
|                 cout<<"Introduzca una cadena: ";
|                 cin>>cad2;

Al hacer el cin tendrias que comprobar que el tama�o de la cadena leida sea menor que la
zona de memoria reservada para cad2.

|                 strcat(cad1,cad2); //aparentemente es en esta l�nea
|                 //en donde falla.

Al hacer el strcat tendr�as que tener en cuenta que no se sobrepasase el maximo que has
reservado previamente para cad1.

Por cierto, al  haber reservado memoria con malloc, lo suyo ser�a que la liberaras con
free (sobre todo si el programa va a ser m�s complicado y va a reservar mucha memoria de
forma din�mica). Si te has decidido a usar new, para liberar la memoria hay que usar
delete.

Saludos

Oscar ........................................