c - リンクリストを使用したインフィックスからポストフィックスへの実装

Question

私は長い間このプログラムをデバッグしようとしてきました。a + b - corのような式を入力するとa / b + c、最初の演算子が2番目の演算子よりも優先順位が高いか等しい場合に正常に機能します。a - b / cただし、最初の演算子の優先順位が 2 番目の演算子よりも低いような式の場合、コンパイラはブレークポイントをスローします。

struct stack
{
    char ele;
    struct stack *next;
};

void push(int);
int pop();
int precedence(char);

struct stack *top = NULL;

int main()
{
    char infix[20], postfix[20];
    int i = 0, j = 0;

    printf("ENTER INFIX EXPRESSION: ");
    gets(infix);

    while(infix[i] != '\0')
    {
        if(isalnum(infix[i]))
            postfix[j++] = infix[i];
        else
        {
            if(top == NULL)
                push(infix[i]);

            else
            {
                while( top != NULL && 
                      (precedence(top->ele) >= precedence(infix[i])) ) 
                    postfix[j++]=pop();
                push(infix[i]);
            }
        }

        ++i;
    }

    while(top != NULL)
        postfix[j++] = pop();

    postfix[j] = '\0';
    puts(postfix);
    getchar();
    return 0;
}

int precedence(char x)
{
    switch(x)
    {
    case '^': return 4;
    case '*':
    case '/': return 3;
    case '+':
    case '-': return 2;

    default: return 0;
    }
}

void push(int x)
{
    int item;
    struct stack *tmp;

    if(top == NULL)
    {
        top = (struct stack *)malloc(sizeof(struct stack));
        top->ele = x;
        top->next = NULL;
    }

    else
    {
        tmp = top;
        top->ele = x;
        top->next = tmp;
    }
}

int pop()
{
    struct stack *tmp;
    int item;

    if(top == NULL)
        puts("EMPTY STACK");

    else if(top->next == NULL)
    {
        tmp = top;
        item = top->ele;
        top = NULL;
        free(tmp);
    }

    else
    {
        tmp = top;
        item = top->ele;
        top = top->next;
        free(tmp);
    }

    return item;
}

私のコーディングを改善する方法についてのアドバイスは役に立ちます。