The following produces garbage in the display and I am sure my code the reason for it. But can't figure out why :

unsigned char xdata FixMsg[] = " Elapsed Time :";
unsigned char * FixMsgPtr = & FixMsg[0];

unsigned char xdata ElapsTime;
unsigned char * ElapsTimePtr = & ElapsTime;

unsigned char xdata DispMsg[];
unsigned char * DispMsgPtr = & DispMsg;

void main(void)
{
  while(1){  
  ....
  ....
  ElapsTime++;
  display (strcat(FixMsgPtr, ElapsTimePtr));
  ...
  ...
}}

// LCD routine.
void display (char *string){
  int i;
  wr_lcdReg(0x01);              // Clear the display
  msDelay(5);

  for (i=0; i<=strlen(string)-1; ++i)
   {
    if (i==16)
       wr_lcdReg(0xc0);
    if (i==32)
       wr_lcdReg(0x80);
    wr_lcdDat(string[i]);
   }
}
 

Raghu