53 lines
1.8 KiB
Plaintext
53 lines
1.8 KiB
Plaintext
BASH PATCH REPORT
|
|
=================
|
|
|
|
Bash-Release: 4.2
|
|
Patch-ID: bash42-028
|
|
|
|
Bug-Reported-by: Mark Edgar <medgar123@gmail.com>
|
|
Bug-Reference-ID: <CABHMh_3d+ZgO_zaEtYXPwK4P7tC0ghZ4g=Ue_TRpsEMf5YDsqw@mail.gmail.com>
|
|
Bug-Reference-URL: http://lists.gnu.org/archive/html/bug-bash/2012-03/msg00109.html
|
|
|
|
Bug-Description:
|
|
|
|
When using a word expansion for which the right hand side is evaluated,
|
|
certain expansions of quoted null strings include spurious ^? characters.
|
|
|
|
Patch (apply with `patch -p0'):
|
|
|
|
*** ../bash-4.2-patched/subst.c 2012-03-11 17:35:13.000000000 -0400
|
|
--- subst.c 2012-03-20 19:30:13.000000000 -0400
|
|
***************
|
|
*** 5810,5813 ****
|
|
--- 5810,5823 ----
|
|
if (qdollaratp && ((hasdol && quoted) || l->next))
|
|
*qdollaratp = 1;
|
|
+ /* If we have a quoted null result (QUOTED_NULL(temp)) and the word is
|
|
+ a quoted null (l->next == 0 && QUOTED_NULL(l->word->word)), the
|
|
+ flags indicate it (l->word->flags & W_HASQUOTEDNULL), and the
|
|
+ expansion is quoted (quoted & (Q_HERE_DOCUMENT|Q_DOUBLE_QUOTES))
|
|
+ (which is more paranoia than anything else), we need to return the
|
|
+ quoted null string and set the flags to indicate it. */
|
|
+ if (l->next == 0 && (quoted & (Q_HERE_DOCUMENT|Q_DOUBLE_QUOTES)) && QUOTED_NULL(temp) && QUOTED_NULL(l->word->word) && (l->word->flags & W_HASQUOTEDNULL))
|
|
+ {
|
|
+ w->flags |= W_HASQUOTEDNULL;
|
|
+ }
|
|
dispose_words (l);
|
|
}
|
|
|
|
*** ../bash-4.2-patched/patchlevel.h Sat Jun 12 20:14:48 2010
|
|
--- patchlevel.h Thu Feb 24 21:41:34 2011
|
|
***************
|
|
*** 26,30 ****
|
|
looks for to find the patch level (for the sccs version string). */
|
|
|
|
! #define PATCHLEVEL 27
|
|
|
|
#endif /* _PATCHLEVEL_H_ */
|
|
--- 26,30 ----
|
|
looks for to find the patch level (for the sccs version string). */
|
|
|
|
! #define PATCHLEVEL 28
|
|
|
|
#endif /* _PATCHLEVEL_H_ */
|