src/usr.bin/mail/edit.c, branch stable/10

Revert r335693, r335694, r335695 by eadler.

2018-07-31T00:37:25+00:00

They cause mail(1) to crash in certain scenarios.

PR:		230196
Reported by:	Pete French

MFC r302776, r302799:

2018-06-27T04:56:01+00:00

mail(1): Bring some fixes from other BSDs.

- Use varargs properly
- Use pid_t
- Better handling of error conditions on forked jobs.
- Some prototype and warning cleanups.

Fix missing forked job changes from r302776 in wait_child().

MFC r270256:

2014-08-28T18:11:05+00:00

Always check the limits of array index variables before using them.

Obtained from:	DragonFlyBSD

fgets(3) returns a pointer, so compare against NULL, not integer 0.

2012-01-13T06:51:15+00:00

Add __unused. Ansi prototypes.

2010-12-19T16:25:23+00:00

Remove the advertising clause from UCB copyrighted files in usr.bin. This

2010-12-11T08:32:16+00:00

is in accordance with the information provided at
ftp://ftp.cs.berkeley.edu/pub/4bsd/README.Impt.License.Change

Also add $FreeBSD$ to a few files to keep svn happy.

Discussed with:	imp, rwatson

Consistently use FBSDID

2002-06-30T05:25:07+00:00

Add (unsigned char) cast to ctype macros

2001-12-19T21:50:22+00:00

ftell->ftello, fseek->fseeko
File 'newsize' type int->off_t
Add visible (long) cast to fsize() when it called to small one message file and
result is assigned to long.

File positions are off_t nowdays, not long, so:

2001-09-02T14:40:51+00:00

    fseek -> fseeko
    ftell -> ftello

    NOTE: fseek/ftell not works for >long offsets per POSIX:

    [EOVERFLOW] For fseek( ), the resulting file offset would be a value which
    cannot be represented correctly in an object of type long.

    [EOVERFLOW] For ftell ( ), the current file offset cannot be represented
    correctly in an object of type long.

Removed duplicate VCS ID tags, as per style(9).

2001-08-13T14:06:34+00:00