bug-bash
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Bash-3.2 Official Patch 57


From: Chet Ramey
Subject: Bash-3.2 Official Patch 57
Date: Sun, 5 Oct 2014 19:08:15 -0400

                             BASH PATCH REPORT
                             =================

Bash-Release: 3.2
Patch-ID: bash32-057

Bug-Reported-by:        Michal Zalewski <lcamtuf@coredump.cx>
Bug-Reference-ID:
Bug-Reference-URL:

Bug-Description:

A combination of nested command substitutions and function importing from
the environment can cause bash to execute code appearing in the environment
variable value following the function definition.

Patch:

*** ../bash-3.2.56/builtins/evalstring.c        2014-09-16 19:08:02.000000000 
-0400
--- builtins/evalstring.c       2014-10-04 15:58:35.000000000 -0400
***************
*** 44,47 ****
--- 44,48 ----
  #include "../redir.h"
  #include "../trap.h"
+ #include "../bashintl.h"
  
  #if defined (HISTORY)
***************
*** 235,244 ****
              struct fd_bitmap *bitmap;
  
!             if ((flags & SEVAL_FUNCDEF) && command->type != cm_function_def)
                {
!                 internal_warning ("%s: ignoring function definition attempt", 
from_file);
!                 should_jump_to_top_level = 0;
!                 last_result = last_command_exit_value = EX_BADUSAGE;
!                 break;
                }
  
--- 236,258 ----
              struct fd_bitmap *bitmap;
  
!             if (flags & SEVAL_FUNCDEF)
                {
!                 char *x;
! 
!                 /* If the command parses to something other than a straight
!                    function definition, or if we have not consumed the entire
!                    string, or if the parser has transformed the function
!                    name (as parsing will if it begins or ends with shell
!                    whitespace, for example), reject the attempt */
!                 if (command->type != cm_function_def ||
!                     ((x = parser_remaining_input ()) && *x) ||
!                     (STREQ (from_file, 
command->value.Function_def->name->word) == 0))
!                   {
!                     internal_warning (_("%s: ignoring function definition 
attempt"), from_file);
!                     should_jump_to_top_level = 0;
!                     last_result = last_command_exit_value = EX_BADUSAGE;
!                     reset_parser ();
!                     break;
!                   }
                }
  
***************
*** 302,306 ****
  
              if (flags & SEVAL_ONECMD)
!               break;
            }
        }
--- 316,323 ----
  
              if (flags & SEVAL_ONECMD)
!               {
!                 reset_parser ();
!                 break;
!               }
            }
        }
*** ../bash-3.2.56/parse.y      2014-09-30 19:43:22.000000000 -0400
--- parse.y     2014-10-04 15:58:35.000000000 -0400
***************
*** 2125,2128 ****
--- 2125,2138 ----
  }
  
+ char *
+ parser_remaining_input ()
+ {
+   if (shell_input_line == 0)
+     return 0;
+   if (shell_input_line_index < 0 || shell_input_line_index >= 
shell_input_line_len)
+     return '\0';      /* XXX */
+   return (shell_input_line + shell_input_line_index);
+ }
+ 
  #ifdef INCLUDE_UNUSED
  /* Back the input pointer up by one, effectively `ungetting' a character. */
*** ../bash-3.2.56/shell.h      2008-04-28 22:00:24.000000000 -0400
--- shell.h     2014-10-04 15:58:35.000000000 -0400
***************
*** 161,164 ****
--- 161,166 ----
  
  /* Let's try declaring these here. */
+ extern char *parser_remaining_input __P((void));
+ 
  extern sh_parser_state_t *save_parser_state __P((sh_parser_state_t *));
  extern void restore_parser_state __P((sh_parser_state_t *));
*** ../bash-3.2/patchlevel.h    Thu Apr 13 08:31:04 2006
--- patchlevel.h        Mon Oct 16 14:22:54 2006
***************
*** 26,30 ****
     looks for to find the patch level (for the sccs version string). */
  
! #define PATCHLEVEL 56
  
  #endif /* _PATCHLEVEL_H_ */
--- 26,30 ----
     looks for to find the patch level (for the sccs version string). */
  
! #define PATCHLEVEL 57
  
  #endif /* _PATCHLEVEL_H_ */

-- 
``The lyf so short, the craft so long to lerne.'' - Chaucer
                 ``Ars longa, vita brevis'' - Hippocrates
Chet Ramey, ITS, CWRU    chet@case.edu    http://cnswww.cns.cwru.edu/~chet/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]