有段代码：在头结点为*p的单链表的第i个位置上插入数据域为x的结点
void insnode(SNode*p,Elemtype x,int i)
{
    SNode *s,*q;
    s=(SNode *)malloc(sizeof(SNode));//为新结点申请空间
     s->data=x;
...
...
...
}

定义代码的时候不分配空间吗？
为什么又要分配空间？

你只是定义了一个指针变量而已，需要动态分配内存空间
如果：SNode s，那么这样你定义一个SNode的变量，才算分配了空间，当然这是在编译时就分配好的