[PATCH MIPS] Work around Bash 4.2 bug

Maciej W. Rozycki macro@imgtec.com
Mon Oct 3 09:50:00 GMT 2016


On Fri, 30 Sep 2016, Hans-Peter Nilsson wrote:

> > The patch below works around the Bash 4.2 bug described at
> > <http://ftp.gnu.org/gnu/bash/bash-4.2-patches/bash42-007>.
> 
> > 	* emulparams/elf32bmipn32-defs.sh: Shift quote of
> > 	"x$EMULATION_NAME" to the left to work around
> > 	<http://ftp.gnu.org/gnu/bash/bash-4.2-patches/bash42-007>.
> >
> 
> >
> > -case x"$EMULATION_NAME" in
> > +case "x$EMULATION_NAME" in
> >  xelf32*n32*) ELFSIZE=32 ;;
> 
> Random comment from the sideline: pretty please add a comment
> regarding the bug workaround *on top of the actual changed code*
> (remember: the "why" goes in the code, not in the changelog), so
> it has a better chance of not being inadvertently reverted but
> instead propagating elsewhere.

 CC-ing <gcc@gcc.gnu.org> as this might affect them too.

 Hmm, the shell construct is so common that I think rather than auditing 
all the scripts throughout our tree I'd rather made a `configure' check 
for the buggy shell feature and reject any shell affected at the top level 
and across subdirectories.  This way we won't have to keep an eye too for 
future script changes which might reintroduce the construct elsewhere.

 Nick, WDYT?

 Also for the purpose of such a check what does x"$EMULATION_NAME" expand 
to in the buggy case given a specific value of $EMULATION_NAME?

  Maciej



More information about the Binutils mailing list