AT>> NO!  Never!  Bad!    You've allocated 40 bytes,
AT>> but the user could easily type 4000.
MM> okay, hows this piece of code ??
MM> char *s;
MM> gets(s);
^^^^^^^^^^^^^^^^^^^^^^
That's the problem.
fgets (s, 40, stdin);
Use that.  It'll only take the first (40-1) characters.
--- PointEd 2.0
---------------