#include "stdio.h"
#include "malloc.h"
#define stack_size 10
#define error
#define ok
typedef char elemtype;
typedef struct stack
{
  elemtype elem[stack_size];
  int top;
  int length;
 }sqstack;

void emptystack(sqstack *s)/*置空栈*/
{

  s->top=-1;

 }

int leng(sqstack *s)
{
  return (s->length);
 }


int push(sqstack *s,elemtype e)/*进栈*/
{
  if(s->top>=stack_size-1)
     return error;
  else
  {
     s->top++;
     s->elem[s->top]=e;
     return ok;
   }
 }

/*int pop(sqstack *s,elemtype *e)
{
  if(s->top<0)
     return error;
  else
  {
    *e=s->elem[s->top];
    s->top--;
    return ok;
   }
 }*/




int main()
{
 sqstack s;
 emptystack(&s);
 push(&s,'a');
 push(&s,'b');
 printf("%d",leng(&s));
 getch();


 }

int push(sqstack *s,elemtype e)/*进栈*/
{
  if(s->top>=stack_size-1)
     return error;
  else
  {
     s->top++;
     s->elem[s->top]=e;
     return ok;
   }
}

因为你的push的时候，length 并不增长，他一直都没有被初始化

为什么你要一个top，同时又要一个length？两个数据不是冗余了吗？
用一个就可以控制该栈了