-> Don't use uninitialized pointers.  The line "char *StrName;" is an
-> uninitialized pointer.  The line "char *StrName[some integer];" is an
-> initialized pointer.  Or you could use
-> {
-> char *StrName;
-> StrName = (char *)malloc(some_integer);
-> }

JK>Only problem is, I don't see how this is going to help you in case of a
JK>string that you simply don't know in advance what size it will be.  It's a
JK>Catch-22; either you must get the string first [gets(string)] and then
JK>malloc(sizeof(string)), which isn't really going to help, or you malloc a
JK>size for your string first, then gets(string) and hope it fits, which isn't
JK>any better than char StrName[some integer].

How about
{
  char *StrName;
  StrName = (char *)malloc(4096);
  gets(StrName);
  StrName = (char *)realloc(StrName, strlen(StrName) + 1);
}
where you initially give it a humongous buffer, fill the buffer, find
the length of the string, add 1 to it to make room for the null byte at
the end, then realloc it down to fit.
___
 * MR/2 2.0 #99 * Internet: bob.liles{at}tpl-bbs.com   RIME: ->7

--- QScan v1.141b / 01-0169