From f3c0b148f29467223ad97d0e09a282079fec9c76 Mon Sep 17 00:00:00 2001 From: joerg Date: Thu, 16 Apr 2015 11:40:43 +0000 Subject: Don't use memcpy for overlapping ranges. Bump revision. --- devel/bmake/Makefile | 3 ++- devel/bmake/files/job.c | 23 ++++++++--------------- 2 files changed, 10 insertions(+), 16 deletions(-) (limited to 'devel') diff --git a/devel/bmake/Makefile b/devel/bmake/Makefile index 2c0f7694dca..329f3f6a7cd 100644 --- a/devel/bmake/Makefile +++ b/devel/bmake/Makefile @@ -1,6 +1,7 @@ -# $NetBSD: Makefile,v 1.67 2014/03/14 21:59:34 ryoon Exp $ +# $NetBSD: Makefile,v 1.68 2015/04/16 11:40:43 joerg Exp $ DISTNAME= bmake-20140314 +PKGREVISION= 1 CATEGORIES= devel MASTER_SITES= # empty DISTFILES= # empty diff --git a/devel/bmake/files/job.c b/devel/bmake/files/job.c index 919f758b1d4..a8179588605 100644 --- a/devel/bmake/files/job.c +++ b/devel/bmake/files/job.c @@ -1,4 +1,4 @@ -/* $NetBSD: job.c,v 1.13 2013/02/26 00:50:14 christos Exp $ */ +/* $NetBSD: job.c,v 1.14 2015/04/16 11:40:43 joerg Exp $ */ /* * Copyright (c) 1988, 1989, 1990 The Regents of the University of California. @@ -70,14 +70,14 @@ */ #ifndef MAKE_NATIVE -static char rcsid[] = "$NetBSD: job.c,v 1.13 2013/02/26 00:50:14 christos Exp $"; +static char rcsid[] = "$NetBSD: job.c,v 1.14 2015/04/16 11:40:43 joerg Exp $"; #else #include #ifndef lint #if 0 static char sccsid[] = "@(#)job.c 8.2 (Berkeley) 3/19/94"; #else -__RCSID("$NetBSD: job.c,v 1.13 2013/02/26 00:50:14 christos Exp $"); +__RCSID("$NetBSD: job.c,v 1.14 2015/04/16 11:40:43 joerg Exp $"); #endif #endif /* not lint */ #endif @@ -142,6 +142,7 @@ __RCSID("$NetBSD: job.c,v 1.13 2013/02/26 00:50:14 christos Exp $"); #include #include "wait.h" +#include #include #include #if !defined(USE_SELECT) && defined(HAVE_POLL_H) @@ -1892,18 +1893,10 @@ end_loop: (void)fflush(stdout); } } - if (i < max - 1) { - /* shift the remaining characters down */ - (void)memcpy(job->outBuf, &job->outBuf[i + 1], max - (i + 1)); - job->curPos = max - (i + 1); - - } else { - /* - * We have written everything out, so we just start over - * from the start of the buffer. No copying. No nothing. - */ - job->curPos = 0; - } + assert(i < max); + /* shift the remaining characters down */ + (void)memmove(job->outBuf, &job->outBuf[i + 1], max - (i + 1)); + job->curPos = max - (i + 1); } if (finish) { /* -- cgit v1.2.3