programa dejo de funciona (lista)

[nandaarea]

Hola tengo una duda cuando meto la siguiente combinacion en mi programa :
elijo opcion 1 guardo numero 2 vuelvo a elejir opcion 1 y guardo numero 1 se rompe alguien me puede decir ¿por que?
aqui les pongo el codigo

Código C:

#include<stdio.h>
#include<stdlib.h>
struct nodo{
       int dato;
       nodo *siguiente;
       };
int menu();
void *crear(void *);
void *eliminar(void *);
void *mostrar(void *);
main(){
       void *p=NULL;
       int x;
       do{
          x=menu();
          switch(x){
                   case 1: p=crear(p);
                           continue;
                   case 2: p=eliminar(p);
                           break;
                   case 3: p=mostrar(p);
                           continue;        
                   } 
         }while(x<4);
       }
int menu(){
    int x;
    do{
        printf("\n\tMenu\n");
        printf("1.- Introducir elemento a la Lista\n");
        printf("2.- Eliminar elemento de la Lista\n");
        printf("3.- Mostar elementos de la Lista\n");
        printf("4.- Salir\n");
        printf("Elige una Opcion => ");
        scanf("%d",&x);
    }while(x<1&&x>4);
    return x;
  }
void *crear(void *p){
     nodo *q,*aux,*aux1;
     q=(nodo*)malloc(sizeof(nodo));
     q->siguiente=NULL;
     printf("Dame un numero para ingresar en la Lista\t"); 
     scanf("%d",&q->dato);
     if(p==NULL){
        p=q;
       }
     else{
          aux=(nodo*)p;
          if(q->dato<aux->dato){
                  q->siguiente=aux;
                  p=q;              
               }
          else{
               aux=(nodo*)p;
               while(aux->dato<q->dato&&aux->siguiente!=NULL){
                     aux1=aux;
                     aux=aux->siguiente;
                     }
               }
          if(aux->siguiente==NULL&&aux->dato<q->dato){
             aux->siguiente=q;                                         
             }
          else{
               aux1->siguiente=q;
               q->siguiente=aux;
               }
          }
     return (p);
     }
void *eliminar(void *p){
     nodo *q,*aux,*aux1;
     int x;
     aux=(nodo*)p;
     printf("Numero a eliminar de la Lista\t");
     scanf("%d",&x);
     if(p==NULL){
                 printf("Lista Vacia");
                 }
     else{
          while(aux->dato<x&&aux->siguiente!=NULL){
               aux1=aux;
               aux=aux->siguiente;                                    
               }
          if(aux->dato==x){
             if(aux==p){
                 if(aux==NULL){
                    p=NULL;           
                    }
                  else{
                       p=aux->siguiente;
                       }       
                 }
              else{
                   if(aux->siguiente==NULL){
                      aux1->siguiente=NULL;                      
                      }
                   else{
                        aux1=aux->siguiente;
                        }
                   }
              free(aux);        
             }
          else{
               printf("Elemento no existente");
               }
          }
     return p;
     }
void *mostrar(void *p){
     nodo*s;
     s=(nodo*)p;
     if(p==NULL){
        printf("\nPila Vacia\n");
        }
     else{
          printf("\n");
          do{
             printf("%d.. ",s->dato);
             s=s->siguiente;
            }while(s!=NULL);
         }
     return p;
     }
 

[vosk]

"...alguien me puede decir ¿por que?..."

Porque aux1 no está definido, y como no lo inicializaste a nulo no tienes forma de hacer ninguna comprovacion de error. Cuando entras 2 y 1 se cumple que if(q->dato < aux->dato), pero ahi no asignas nada a aux1, sino que solo asignas aux1 en el siguiente else pero que en este caso no entras:

Código:

if(p==NULL){
		p=q;
	}
	else {//en la segunda llamada entras aqui
		aux = (struct nodo*)p;
		if(q->dato < aux->dato) {//se cumple esto
			q->siguiente=aux;
			p=q;              
		}
		else {//y no se ejecuta esto
			aux=(struct nodo*)p;
			while(aux->dato<q->dato&&aux->siguiente!=NULL) {
				aux1=aux;//por lo tanto no asignas nada a aux1
				aux=aux->siguiente;
			}
		}
		
		if(aux->siguiente == NULL && aux->dato < q->dato) {
			aux->siguiente = q;                                       
		}
		else {//aqui aux1 contiene datos basura
			aux1->siguiente = q;
			q->siguiente = aux;
		}
	}

Saludos
vosk