G'Day All,
 
-=> Quoting Frank Adam to All <=-

I 've made a couple of booboos here, there is a potential memory leak,
and the strcpy is an unnecessary overhead.

 FA> ** char* In_Str(char *buffer,char *word)
 FA> ** Public domain by Frank Adam, and so on...

 FA> char *txt;char *pat;char *ptr;
 FA> txt = (char*) malloc(strlen(buf)+1);
 FA> pat = (char*) malloc(strlen(wrd)+1);
 FA> if(!txt || !pat) return NULL;
This would be a leak if(txt && !pat).

 FA> strcpy(txt,buf); strcpy(pat,wrd);     /* leave the originals alone */
This was wasteful..but easy :-)

So here is the revision..

char* In_Str(char *buf,char *wrd)  /* case insensitive strstr() */
{
 char *txt;char *pat;char *ptr,*ptr1;
 txt = (char*) malloc(strlen(buf)+1);
 if(!txt) return NULL;
 pat = (char*) malloc(strlen(wrd)+1);
 if(!pat) {free(txt); return NULL;}
 ptr = buf;ptr1 = txt; while(*ptr++) *ptr1++ = tolower(*ptr);
 ptr = wrd;ptr1 = pat; while(*ptr++) *ptr1++ = tolower(*ptr);
 ptr = strstr(txt,pat);
 if(ptr) ptr = buf+(ptr-txt);
 free(txt);free(pat);
 return ptr;
}


  L8r Frank (fadam{at}ozemail.com.au).
  
___ Blue Wave/DOS v2.21

--- Maximus 3.01