diff options
| author | Eduardo Chappa <chappa@washington.edu> | 2018-01-09 12:07:31 -0700 |
|---|---|---|
| committer | Eduardo Chappa <chappa@washington.edu> | 2018-01-09 12:07:31 -0700 |
| commit | f2246f7d298b96a231dcde60cf62711ca22838ed (patch) | |
| tree | 4bd3d835b0b93ae18f3922c039a1ee24262cb157 | |
| parent | 30507e11007fbacb942016b0a1167104ad2c7d49 (diff) | |
| download | alpine-f2246f7d298b96a231dcde60cf62711ca22838ed.tar.xz | |
* Change call from strncpy to memmove in pith/reply.c, in the
forward_subject function, as this causes problems with
overlapping memory. The problem exists in slackware 12.4.
Reported by Rich Shepard.
| -rw-r--r-- | pith/pine.hlp | 2 | ||||
| -rw-r--r-- | pith/reply.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/pith/pine.hlp b/pith/pine.hlp index 6fb5e1bb..2bb809a9 100644 --- a/pith/pine.hlp +++ b/pith/pine.hlp @@ -140,7 +140,7 @@ with help text for the config screen and the composer that didn't have any reasonable place to be called from. Dummy change to get revision in pine.hlp ============= h_revision ================= -Alpine Commit 242 2018-01-07 20:51:03 +Alpine Commit 243 2018-01-09 12:05:19 ============= h_news ================= <HTML> <HEAD> diff --git a/pith/reply.c b/pith/reply.c index 4c0a6f44..383294ec 100644 --- a/pith/reply.c +++ b/pith/reply.c @@ -2236,7 +2236,7 @@ forward_subject(ENVELOPE *env, int flags) (l < 5 || strcmp(tmp_20k_buf+l-5,"(fwd)"))){ snprintf(tmp_20k_buf+2000, SIZEOF_20KBUF-2000, "%s (fwd)", tmp_20k_buf); tmp_20k_buf[SIZEOF_20KBUF-2000-1] = '\0'; - strncpy(tmp_20k_buf, tmp_20k_buf+2000, SIZEOF_20KBUF); + memmove(tmp_20k_buf, tmp_20k_buf+2000, SIZEOF_20KBUF); tmp_20k_buf[SIZEOF_20KBUF-1] = '\0'; } |