Bug 214973 - bmake segfault on parenthesized variables.
Summary: bmake segfault on parenthesized variables.
Status: New
Alias: None
Product: Base System
Classification: Unclassified
Component: bin (show other bugs)
Version: 11.0-STABLE
Hardware: Any Any
: --- Affects Many People
Assignee: freebsd-toolchain (Nobody)
Keywords: patch
Depends on:
Reported: 2016-12-01 04:27 UTC by Ori Bernstein
Modified: 2016-12-10 03:28 UTC (History)
0 users

See Also:

Fixes segfault in bmake. Possibly sketchy. (435 bytes, patch)
2016-12-01 04:27 UTC, Ori Bernstein
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ori Bernstein 2016-12-01 04:27:40 UTC
Created attachment 177565 [details]
Fixes segfault in bmake. Possibly sketchy.

Turns out that I can trivially segfault make with this input:


This happens because in /usr/src/contrib/bmake/parse.c:1862 or
so, we start off with:

	for (depth = 0, cp = line + 1; depth > 0 || *cp != '='; cp++) {

which skips over the opening '(', meaning that when we see the closing ')',
the depth becomes negative, and we never break out of the loop, eventually
reading outside of mapped memory.

Starting off with 'cp = line', as in the attached patch, seems to work,
although I'm a bit suspicious about it breaking some subtle case when
parsing variables.

Still, I tested by:

	cd /usr/src/usr.bin/bmake; make; make install
	cd /usr/src/lib/libc; make clean; make
Seems to work. 'make world' is running now.