求大神看一下用链栈判断回文串的C语言代码哪里不对，帮忙改一下

#include <stdio.h>
#include <string.h>
#include <malloc.h>
typedef struct node
{
char data;
struct node *link;
}JD;

JD *push(JD *top,char x)
{
JD *p;
p=(JD*)malloc(sizeof(JD));
p->data=x;
p->link=top;
top=p;
return(p);
}
JD *pop(JD *top,char *p)
{
JD *q;
if(top!=NULL)
{
q=top;
*p=top->data;
top=top->link;
free(q);
}
return top;
}
int main()
{
JD *top;
top=(JD*)malloc(sizeof(JD));
top->link=NULL;
int i,lenth;
char str[20]="abcdcba";
char e;
lenth=strlen(str);
for(i=0; i<lenth; i++)
push(top,str[i]);
for(i=0;i<lenth;i++)
{
pop(top,&e);
if(e!=str[i])
{
printf("NO");
return;
}
}
printf("YES");
}