[PATCH v2 1/2] ash: introduce ash_perror_msg_and_raise_error()

Johannes Schindelin Johannes.Schindelin at gmx.de
Tue Jul 25 10:39:57 UTC 2017


Hi Ron,

On Sun, 23 Jul 2017, Ron Yorston wrote:

> Johannes Schindelin wrote:
> >@@ -1263,6 +1263,8 @@ ash_vmsg(const char *msg, va_list ap)
> > 			fprintf(stderr, "line %d: ", startlinno);
> > 	}
> > 	vfprintf(stderr, msg, ap);
> >+	if (strerr)
> >+		vfprintf(stderr, ": ", strerr);
> > 	newline_and_flush(stderr);
> > }
> > 
> 
> I think that should be:
> 
> >+	if (strerr)
> >+		fprintf(stderr, ": %s", strerr);

Oops, you're right! Funny how GCC did not catch that error.

Thanks,
Johannes


More information about the busybox mailing list