summaryrefslogtreecommitdiffstats
path: root/CWRU/changelog
diff options
context:
space:
mode:
Diffstat (limited to 'CWRU/changelog')
-rw-r--r--CWRU/changelog4967
1 files changed, 4967 insertions, 0 deletions
diff --git a/CWRU/changelog b/CWRU/changelog
new file mode 100644
index 0000000..c33023f
--- /dev/null
+++ b/CWRU/changelog
@@ -0,0 +1,4967 @@
+ 9/15/2016
+ ---------
+[bash-4.4 released]
+
+ 9/17
+ ----
+builtins/common.c
+ - display_signal_list: if 0 is supplied as an argument to kill -l,
+ display a signal-out-of-range error. Fixes bug reported by
+ Martijn Dekker <martijn@inlv.org>
+
+ 9/18
+ ----
+print_cmd.c
+ - print_heredoc_bodies: take a list of redirections, all here documents,
+ and print the here-document bodies including the ending delimiter,
+ followed by a newline
+ - print_redirection_list: print the heredoc headers the first time
+ through the list of redirections so we print them in the same left-
+ to-right order they're supplied; print the bodies and delimiters with
+ print_heredoc_bodies after we print all the redirections if we're
+ not printing a list of commands (printing_connection == 0). Fix for
+ problem reported by Isabella Parakiss <izaberina@gmail.com>
+ - print_deferred_heredocs: don't print heredoc headers, since they're
+ now printed in print_redirection_list
+ - print_deferred_heredocs: use print_heredoc_bodies
+
+ 9/20
+ ----
+builtins/mapfile.def
+ - readarray: update usage synopsis to include -d delim. Report and
+ fix from jhankins@homewood.k12.al.us
+
+test.c
+ - test_binop: if op[0] == '-', make sure op[1] != 0 before checking
+ op[2] or op[3]. Fixes out-of-bounds read reported by Franco
+ Costantini <franco.costantini.20@gmail.com>
+
+parse.y
+ - cond_term: if yylval.word->word[0] == '-' make sure word[1] != 0
+ before checking word[2] and calling test_unop. Fixes out-of-bounds
+ read reported by Franco Costantini <franco.costantini.20@gmail.com>
+ - make sure to check all instances of the return value of the
+ simple_command production before trying to hook redirections onto it,
+ since it can potentially return NULL.
+ Reported by Franco Costantini <franco.costantini.20@gmail.com>
+
+ 9/21
+ ----
+parse.y
+ - token_is_assignment: make sure to check that the return value from
+ assignment() finds the `=' we just added at index I to avoid false
+ positives from g=g=g=g=g=g=(). Reported by Franco Costantini
+ <franco.costantini.20@gmail.com>
+
+ 9/22
+ ----
+parse.y
+ - token_is_assignment: if we're not using the bash malloc (which has
+ guard bytes after the requested allocation size), use a new
+ buffer so we can make sure it's big enough to hold the current
+ token and at least two additional characters. Reported by
+ Franco Costantini <franco.costantini.20@gmail.com>
+
+ 9/23
+ ----
+execute_command.c
+ - execute_in_subshell: call without_job_control for both job control
+ and non-job-control builds to avoid waiting for last_made_pid
+ inherited from parent shell. Related to fix from 8/4 to
+ initialize_subshell. Fixes bug reported by Martijn Dekker
+ <martijn@inlv.org>
+
+builtins/declare.def
+ - local_builtin: allow `local --help' to work even when not executing
+ a function. Suggestion from Norm Abramovitz <norm@starkandwayne.com>
+
+ 9/25
+ ----
+builtins/printf.def
+ - bexpand: if passed a null or empty string, return a newly-allocated
+ empty string (so the return value can always be freed) instead of
+ NULL.
+ - printstr: if passed a null string, treat it as if it were an empty
+ string so any precision or fieldwidth arguments are handled
+ correctly. Report from isabella parakiss <izaberina@gmail.com>
+
+subst.c
+ - skip_matched_pair: make the flags argument of 1 affect whether or
+ not backslash can quote characters in the string, so it affects
+ all quoting mechanisms
+
+arrayfunc.c
+ - unbind_array_element: now takes a third (flags) argument; flags&1
+ means to not expand an associative array index. This is only called
+ from a shell builtin context, which means the argument should have
+ already undergone word expansion, so it should be ok. Will still
+ be a problem for uses like "unset -v 'var[$ind]'", quoted to avoid
+ globbing
+ - valid_array_reference: pass second flags argument to skipsubscript;
+ still set to 0 by all callers
+
+arrayfunc.h
+ - unbind_array_element: updated prototype
+
+builtins/set.def
+ - unset_builtin: add third argument of 1 to unbind_array_element calls
+ [TEMPORARILY DISABLED]
+ - unset_builtin: call valid_array_reference with second arg of 1 to
+ handle unmatched pairs of quotes and [/]
+
+array.c
+ - array_reference: optimize access locality as well as sequential
+ access by adding ability to start from last-accessed element and move
+ backwards or forwards depending on the desired index. Inspired by
+ report from Tom McCurdy <tom.j.mccurdy@gmail.com> and followup from
+ Christian Franke <Christian.Franke@t-online.de>
+
+variables.c
+ - all_array_variables: include associative arrays, not just indexed
+ arrays. Reported by Grisha Levit <grishalevit@gmail.com>
+
+execute_cmd.c
+ - execute_disk_command: if we're optimizing out the fork (CMD_NO_FORK),
+ decrement the shell level so we don't end up incrementing it twice.
+ We should decrement the shell level for an implicit exec like an
+ explicit one. Reported by Stephane Chazelas
+ <stephane.chazelas@gmail.com>
+
+builtins/exec.def
+ - exec_builtin: don't bother to decrement the shell level if we are
+ already in a `(command)' subshell. Reported by Stephane Chazelas
+ <stephane.chazelas@gmail.com>
+
+lib/readline/macro.c
+ - macro_level: count of macro nesting level, maintained by
+ _rl_push_executing_macro and _rl_pop_executing macro
+ - _rl_with_macro_input: if current nesting level exceeds
+ MAX_MACRO_LEVEL, abort the current command line with _rl_abort_internal
+ and return to the top level. Reported by Christian Klomp
+ <christianklomp@gmail.com>
+
+lib/readine/history.c
+ - add_history: if allocating the history list for the first time,
+ make sure the max history list size isn't so large that it will
+ cause allocation errors. Cap it at MAX_HISTORY_INITIAL_SIZE
+ (8192). Reported by Sean Zha <freeman_cha@hotmail.com>
+
+jobs.c
+ - wait_for: if a foreground job with job control enabled is killed by a
+ SIGINT and sourcelevel != 0, act as if we received the SIGINT so the
+ file sourcing can be terminated. Reported by PePa
+ <peterkelly@passchier.net>
+
+ 9/26
+ ----
+subst.c
+ - read_comsub: only warn once for null bytes in command substitution
+ output, instead of once for every null byte
+
+arrayfunc.c
+ - array_variable_part: now takes a `flags' argument; changed all callers
+ to initially pass 0
+ - array_variable_name: now takes a `flags' argument; changed all callers
+ to initially pass 0
+ - array_variable_name: pass `flags' argument to skipsubscript instead
+ of just passing 0
+
+arrayfunc.h
+ - array_variable_{name,part}: added new argument to prototypes
+
+ 9/29
+ ----
+bashline.c
+ - bash_directory_completion_hook: don't bother to try and expand a
+ ${ or $( that isn't complete; expand_prompt_string will complain.
+ Fixes reports from John Passaro <john.a.passaro@gmail.com> and
+ Valentin Bajrami <valentin.bajrami@gmail.com>
+
+lib/readline/history.c
+ - _hs_append_history_line: use a strategy that attempts to avoid
+ realloc copying memory to a newly-allocated block if the history
+ line we're appending to gets `too long' (currently 256 bytes).
+ We reallocate in powers of 2 starting at 512, and rely on realloc
+ not allocating a new block and copying into it if the size is the
+ same as the last call to realloc. Addresses issue raised by
+ Hubert Schmid <h.schmid@gmx.de>
+
+trap.h
+ - check_signals: new extern declaration
+ - trapped_signal_received: new extern declaration
+
+builtins/read.def
+ - read_builtin: if one of the zread* functions returns < 0 (which
+ usually only happens in Posix mode), make sure we call check_signals()
+ before eventually calling run_pending_traps() because zread() isn't
+ going to call it
+ - include trap.h for correct extern functions and variable declarations
+
+lib/sh/zread.c
+ - zread: if executing a builtin, call check_signals_and_traps() for
+ backwards compatibility. If not, call check_signals() only, because
+ we don't want to run traps during, for instance, reading command
+ substitution output. Fixes race condition bug reported by Luiz Angelo
+ Daros de Luca <luizluca@gmail.com>
+
+ 9/30
+ ----
+subst.c
+ - command_substitute: reset parse_and_execute_level to 0 in the child
+ process, since it's independent of the other parse_and_execute
+ calls. Adds command substitution inside other parse_and_execute
+ calls optimizations to suppress forks, as suggested by
+ Martijn Dekker <martijn@inlv.org>
+
+ 10/3
+ ----
+configure.ac
+ - SHOBJ_STATUS: make sure it defaults to unsupported and is substituted
+ if the shobj-conf script isn't run. Fixes `make install' bug with
+ a minimal config reported by Andrew Tomazos <andrewtomazos@gmail.com>
+
+ 10/5
+ ----
+support/shobj-conf
+ - darwin: set compatibility_version for a shared build of the readline
+ library (the standalone readline distribution shares this file) to
+ $(SHLIB_MAJOR)$(SHLIB_MINOR). Recommendation from Max Horn
+ <max@quendi.de>
+
+ 10/6
+ ----
+array.h
+ - array_first_index: new convenience define
+
+array.c
+ - ADD_AFTER: new define, complement of ADD_BEFORE
+ - UNSET_LASTREF: now takes an array as an argument, prepping for move
+ of lastref pointer into the array struct
+ - array_insert: check whether we are adding at the beginning of the
+ array and take a fast path if so
+ - array_insert: use same strategy as array_reference to find the place
+ to insert, starting from the last-referenced element and moving
+ forward or back from there; use ADD_AFTER if moving backward
+ - array_insert: if replacing an existing element, just replace the
+ value with new->value instead of the entire element
+ - array_reference: short-circuit quickly if looking for an element
+ before the first assigned index
+ - array_reference: if we don't find the element, leave lastref pointing
+ to the closest element under the assumption we will be assigning or
+ looking for something close
+ - array_reference: take advantage of ordered indexes to short-circuit
+ when looking for element that is not set
+
+ 10/7
+ ----
+array.c
+ - array_remove: short-circuit if asked to remove index after max
+ index or before first index
+
+ 10/10
+ -----
+lib/malloc/malloc.c
+ - internal_realloc: if we are requesting reallocation to the same size
+ as the block's current size, short-circuit and return `mem' right
+ after doing bounds check
+ - internal_realloc: if we are reducing the size of an allocation, and
+ the new size fits in the next lower bin, just keep the same block
+ and adjust the size, so we can avoid some copies
+
+parse.y
+ - set_line_mbstate: keep track of the allocated size of
+ shell_input_line_property, only request reallocation if the size
+ increases, but don't let it get too big
+
+ 10/11
+ -----
+jobs.c
+ - wait_for_background_pids: make sure we wait for pid in
+ last_procsub_child since it's not found in any job -- still needs
+ more work to wait for multiple process substitutions
+
+subst.c
+ - process_substitute: if make_child fails, make sure we call
+ restore_pipeline to undo the previous save_pipeline()
+
+ 10/15
+ -----
+subst.c
+ - process_substitute,command_substitute: leave subshell_level (reflected
+ as $BASH_SUBSHELL) intact for any exit trap instead of decrementing
+ it. Suggested by Martijn Dekker <martijn@inlv.org>
+
+builtins/evalstring.c
+ - optimize_subshell_command: new function, framework for optimizing
+ out forks for command that have already forked and are executing
+ in a subshell. Not used yet
+
+expr.c
+ - readtok: if we have a post-increment or post-decrement, and the
+ previous token is not a string, check whether the previous token is
+ a number that results from a pre-increment or pre-decrement, and
+ make that an error. Report from Conrad Hoffmann <ch@bitfehler.net>
+
+ 10/16
+ -----
+aclocal.m4
+ - BASH_JOB_CONTROL_MISSING: convert from AC_TRY_RUN to AC_TRY_COMPILE
+ with existing set of preprocessor defines, so it can work when
+ cross-compiling. Suggested by Felix Janda <felix.janda@posteo.de>
+
+ 10/19
+ -----
+variables.c
+ - get_bashpid: BASHPID is no longer readonly; assignments to it are
+ just ignored.
+
+doc/{bash.1,bashref.texi}
+ - BASHPID: note that assignments are ignored and unsetting BASHPID
+ causes it to lose its special properties
+
+ 10/28
+ -----
+builtins/pushd.def
+ - popd_builtin: make sure to check the normalized stack offset
+ (i.e., negatives counting back from the end of the stack) is within
+ bounds before trying to free that stack entry. Report from
+ Fernando Muñoz <fernando@null-life.com>
+
+lib/readline/histfile.c
+ - chown: protect calls with HAVE_CHOWN. Fixes for MinGW{64,32} from
+ J. Peter Mugaas <jpmugaas@suddenlink.net>
+
+lib/readline/colors.c
+ - S_ISDIR: add define if missing
+ - S_ISUID, S_ISGID, S_ISLNK, S_ISSOCK: don't use unless defined.
+ Fixes for MinGW{64,32} from J. Peter Mugaas <jpmugaas@suddenlink.net>
+
+bashhist.c
+ - current_command_line_comment: set to the most recent line in a
+ possibly multi-line command that contains a shell comment; set in
+ maybe_add_history and bash_add_history to either the current
+ command number or -2
+ - maybe_add_history: set current_command_line_comment for first line
+ of command
+ - bash_add_history: chars_to_add set to "\n" if the current command line
+ is one greater than the previous line containing a comment
+ (current_command_line_comment). Fixes complaint from back in
+ January, 2016 from Dave Rutherford <dave@evilpettingzoo.com>
+
+array.h
+ - lastref: move last-referenced pointer into each array struct, so all
+ arrays can have reference locality
+
+array.c
+ - array_create: set lastref element to 0
+ - array_copy: if copying the lastref in array a, set the new lastref
+ in the copy to the same element
+ - IS_LASTREF, LASTREF_START, LASTREF, INVALIDATE_LASTREF, SET_LASTREF,
+ UNSET_LASTREF: change to use new array `lastref' member
+
+pathexp.c
+ - quote_string_for_globbing: if quoting for a regexp, make sure to
+ skip and copy a leading `^' in a bracket expression, and skip and
+ copy a subsequent `]' (after an optional `^'), so that leading
+ bracket doesn't close the bracket expression. Report from
+ Stephane Chazelas <stephane.chazelas@gmail.com>
+
+ 10/29
+ -----
+subst.c
+ - extract_delimited_string: add calls to CHECK_STRING_OVERRUN for
+ $( inside $((, nested OPENERs, nested ALT_OPENERs, and backquotes
+ - skip_to_delim: add calls to CHECK_STRING_OVERRUN for $(, ${,
+ process substitution
+ - extract_dollar_brace_string: add calls to CHECK_STRING_OVERRUN for
+ $(
+ - extract_dollar_brace_string: use skipsubscript to skip over the
+ array subscript in ${var[sub]} (non-quoted case uses string_extract,
+ which already uses skipsubscript). Tagged for bash-5.0
+
+ 10/30
+ -----
+variables.c
+ - EPOCHSECONDS: new dynamic variable, time in seconds since Unix
+ epoch; assignments are ignored. Can be used on systems where
+ strftime() doesn't support '%s'
+ - EPOCHREALTIME: new dynamic variable, time in seconds since Unix
+ epoch with microsecond granularity
+
+doc/{bash.1,bashref.texi}
+ - EPOCHSECONDS documentation
+ - EPOCHREALTIME documentation
+
+execute_cmd.c
+ - decpoint: moved to locale.c, renamed locale_decpoint; changed callers
+
+{bashintl,externs}.h
+ - locale_decpoint: extern declaration or #define if support for
+ localeconv() not there
+
+ 10/31
+ -----
+lib/malloc/malloc.c
+ - posix_memalign: add new posix-mandated interface
+
+examples/loadables/rm.c
+ - rm: minimal loadable builtin, removes files and directories, only
+ handles -r and -f options. Original from Tim Ruehsen
+ <tim.ruehsen@gmx.de>, heavily rewritten for inclusion as loadable
+
+examples/loadables/Makefile.in
+ - rm: add rules to build rm as one of the `other' targets
+
+examples/loadables/stat.c
+ - stat: new loadable builtin that takes a filename and loads the info
+ returned by stat(2) into an associative array specified by the -A
+ argument (default STAT)
+
+ 11/1
+ ----
+variables.c
+ - BASH_ARGV0: new dynamic variable, returns $0 on reference and sets
+ $0 on assignment. From a suggestion from Rocky Bernstein <rocky@gnu.org>
+ a few years ago
+
+doc/{bash.1,bashref.texi}
+ - BASH_ARGV0: document
+
+ 11/2
+ ----
+lib/glob/sm_loop.c
+ - parse_collsym: make sure to not return an out-of-bounds read if a
+ collating symbol is unterminated. Fixes OOB read reported by
+ Jerzy Kramarz <op7ica@gmail.com>
+ - brackmatch: after incrementing p, before checking whether it's a
+ character range, check whether *p was NULL before the increment
+ and short-circuit the bracket expression if it is
+
+jobs.c
+ - discard_last_procsub_child: new function, safely discards
+ last_procsub_child and sets it to NULL
+
+subst.c
+ - process_substitute: call discard_last_procsub_child instead of
+ calling discard_pipeline directly. Fixes bug reported by
+ Christian Weisgerber <naddy@mips.inka.de>
+
+ 11/3
+ ----
+shell.h
+ - EX_DISKFALLBACK: new special return status available to builtins;
+ means to attempt to execute a disk command with the same name as
+ the builtin
+
+examples/loadables/rm.c
+ - if we see the -i option, return EX_DISKFALLBACK
+
+execute_cmd.c
+ - execute_simple_command: if executing a builtin returns EX_DISKFALLBACK,
+ try running execute_disk_command instead
+
+bashline.c
+ - shell_expand_line: use expand_word to expand the readline line
+ buffer, which allows us to pass flags with the word. If a numeric
+ argument is supplied, do not perform quote removal (pass
+ Q_HERE_DOCUMENT since here-doc quoting does the right thing) and
+ do not perform command or process subsitution. From a suggestion
+ by Dabrien 'Dabe' Murphy <dabe@dabe.com> based on an old bug-bash
+ discussion
+
+subst.c
+ - expand_word_internal: note that we have added a quoted IFS char to
+ istring by setting sentinel has_quoted_ifs. Usually we only add
+ one if we are not going to be performing word splitting, but we
+ will not perform word splitting if there's no expansion, so we need
+ to take care of that case
+ - expand_word_internal: when performing final word split, if there are
+ no expansions but has_quoted_ifs is non-zero, call remove_quoted_ifs()
+ to remove any quoted ifs characters we added while processing
+ - remove_quoted_ifs: new function, removes CTLESC chars preceding
+ (single-byte) chars in IFS. Used when we are not performing word
+ splitting. Fixes bug reported by Martijn Dekker <martijn@inlv.org>
+
+ 11/4
+ ----
+lib/readline/macro.c
+ - _rl_peek_macro_key: return the next character from the current
+ keyboard macro; the next character from the `next' keyboard macro,
+ if there is one, if at the end of the current macro; or 0 to
+ indicate that we are at the end of a keyboard macro sequence
+
+lib/readline/rlprivate.h
+ - _rl_peek_macro_key: extern declaration
+
+lib/readline/readline.c
+ - _rl_dispatch_subseq: add test for ESC at the end of a keyboard macro,
+ which should cause the keyboard timeout for ESC to kick in. The
+ previous test didn't run the timeout code if executing from a macro,
+ even if we had read the last character of the macro. Fixes bug
+ reported by Clark Wang <clarkw@vmware.com>
+
+lib/glob/sm_loop.c
+ - GMATCH: allow trailing backslash in pattern to explicitly match a
+ backslash that is the last character in the string. Bug report from
+ Stephane Chazelas <stephane.chazelas@gmail.com>
+
+ 11/5
+ ----
+builtins/common.c
+ - display_signal_list: if displaying a signal name corresponding to an
+ exit status > 128, don't display the SIG prefix at all. Old code
+ made displaying the SIG prefix dependent on JOB_CONTROL define.
+ Report and fix from Martijn Dekker <martijn@inlv.org>
+
+execute_cmd.c
+ - execute_subshell_builtin_or_function: call without_job_control even
+ if JOB_CONTROL is not defined. Similar to fix from 9/23.
+ Report from Martijn Dekker <martijn@inlv.org>
+
+execute_cmd.c
+ - execute_simple_command: free memory allocated and passed to
+ make_child in the child process if JOB_CONTROL is defined
+ - execute_command_internal: free memory allocated and passed to
+ make_child in the child process created to run a () subshell or a
+ compound command within a pipeline if JOB_CONTROL is defined
+ - execute_coproc: free memory allocated and passed to make_child in
+ the child process if JOB_CONTROL is defined
+ - execute_disk_command: free memory allocated and passed to
+ make_child in the child process if JOB_CONTROL is defined. This
+ series of fixes is the result of reports from
+ Eduardo A. Bustamante López <dualbus@gmail.com>
+
+ 11/6
+ ----
+lib/sh/unicode.c
+ - u32toutf16: fix to prevent outputting broken surrogate pairs for
+ Japanese locales (ja_JP.UTF-8) on cygwin (which uses UTF-16 natively).
+ Report and fix from Koichi MURASE <myoga.murase@gmail.com>
+
+builtins/trap.def
+ - trap_builtin: if OP is `-' (revert), set the SIGINT signal handler
+ to sigint_sighandler if the shell is interactive and sourcing a
+ file (interactive_shell && sourcelevel) or running a trap
+ (interactive_shell && running_trap) even if it's not currently
+ interactive. Report from Martijn Dekker <martijn@inlv.org>
+
+builtins/read.def
+ - check for and read multibyte characters in all cases, not just when
+ we are reading a specific number of characters, as long as
+ mb_cur_max > 1
+
+subst.c
+ - expand_word_internal: some improvements to code that converts istring
+ into a WORD_LIST * to avoid multiple allocations and copies of
+ istring, which is already malloc'ed memory -- reduce number of malloc
+ and free calls
+
+test.c
+ - unary_test: make sure if we test -v array[@] or array[*] that we
+ free the return value from array_value
+
+ 11/8
+ ----
+expr.c
+ - expcond: make sure to set `noeval' before reading tokens depending on
+ the result of the conditional test, since readtok() can evaluate
+ identifiers (and recursively those containing expressions). Report
+ and fix from Koichi MURASE <myoga.murase@gmail.com>
+
+builtins/evalstring.c
+ - should_suppress_fork: make sure to check for traps on EXIT and
+ ERR, since any_signals_trapped() only checks for `real' signals,
+ not the fake shell ones. Fixes bug reported by Werner Fink
+ <werner@suse.de>
+ - optimize_subshell_command: ditto
+
+ 11/9
+ ----
+eval.c
+ - reader_loop: change so that we don't reset the SIGINT handler every
+ time through the command loop in an interactive shell if the signal
+ is trapped (as the comment noted). Reported by Report from Martijn
+ Dekker <martijn@inlv.org>
+
+subst.c
+ - parameter_brace_{patsub,remove_pattern,transform,casemod}: save and
+ restore this_command_name while temporarily setting it for use in
+ error messages. Fixes use-after-free error reported by
+ Jerzy Kramarz <op7ica@gmail.com>
+ - string_extract_verbatim: make sure when we increment i by 2 due to
+ a CTLESC or CTLESC-CTLNUL that we don't read past the end of the
+ string. This can happen if the string ends with an odd number of
+ CTLESC chars. Fixes oob-read error reported by
+ Jerzy Kramarz <op7ica@gmail.com>
+
+ 11/11
+ -----
+configure.ac,aclocal.m4
+ - BASH_FUNC_SBRK: make sure sbrk actually works, instead of being just
+ a stub function like on several Linux distributions
+
+lib/malloc/malloc.c
+ - malloc_usable_size: return the maximum number of bytes available for
+ a particular memory allocation (size of block allocated for it)
+
+ 11/12
+ -----
+configure.ac
+ - changes to make --enable-profiling work on Linux (-pg, no static link,
+ no -static in LDFLAGS)
+
+Makefile.in
+ - changes to make --enable-profiling work on Linux (take
+ ${PROFILE_FLAGS} out of BASE_CCFLAGS, add to CCFLAGS instead; add
+ BASE_LDFLAGS define that doesn't include -pg or -static; use
+ BASE_LDFLAGS in LDFLAGS; move ${PROFILE_FLAGS} and ${STATIC_LD} to
+ LDFLAGS)
+
+tests/run-gprof
+ - changes to make profiling test suite with gprof work on Linux (set
+ GMON_OUT_PREFIX to handle multiple profiled processes per test;
+ change default filename to gmon.out from bash.gmon; look for
+ gprof output files in ${TMPDIR} instead of /tmp; handle multiple
+ profiling files starting with $GMON_OUT_PREFIX)
+
+locale.c
+ - locale_isutf8: use locale_charset() if it's available and nl_langinfo
+ isn't
+ - locale_isutf8: add code to parse a locale specification and determine
+ whether the encoding is UTF-8 (or utf8) in the absence of
+ nl_langinfo and locale_charset
+ - set_default_locale: make sure to set locale_utf8locale early on
+ - locale_utf8locale: set via call to locale_isutf8 wherever LANG,
+ LC_ALL, or LC_CTYPE is set
+
+ 11/13
+ -----
+lib/readline/display.c
+ - rl_redisplay: handle report of potential integer overflow in
+ horizontal display mode from Mateusz Lenik <mlen@mlen.pl>
+
+lib/sh/mbschr.c
+ - mbschr: if in a UTF-8 locale (locale_utf8locale) and we are looking
+ for an ASCII character, use (essentially) strchr as a shortcut
+
+lib/sh/shmbchar.c
+ - utf8_{mbstrlen,mblen,mbsmbchar,mbsnlen}: UTF-8-specific replacement
+ functions (should also be able to use utf8_mblen in subst.c:
+ string_extract_verbatim)
+ - mbsmbchar: if locale_utf8locale is true, call utf8_mbsmbchar
+ - mbsmbchar: in UTF-8 locale, use utf8_mblen instead of mbrlen (not
+ used if earlier shortcut taken)
+
+lib/sh/unicode.c
+ - u32cconv: don't use nl_langinfo early to call u32toutf8()
+ - u32cconv: use locale_utf8locale to set utf8locale
+ - u32cconv: when initializing (u32init == 0), use locale_charset, then
+ nl_langinfo, then stub_charset to determine the charset
+ - u32cconv: even if we don't have iconv, if locale_utf8locale is non-
+ zero, return u32toutf8
+
+ 11/15
+ -----
+lib/readline/nls.c
+ - _rl_init_locale: new function, split off code that determines current
+ locale from _rl_init_eightbit, use it to set _rl_utf8locale
+
+lib/readiline/rlprivate.h
+ - _rl_init_locale: new extern declaration
+
+lib/readline/readline.c
+ - rl_initialize: if not initializing everything the first time, call
+ _rl_init_locale to check current locale and set _rl_utf8locale
+
+lib/readline/text.c
+ - _rl_insert_char: optimize cases where we are inserting a single-byte
+ character in a locale with multibyte characters
+ - _rl_insert_char: check whether character is single byte if we know
+ we are in a UTF-8 locale, optimize single-byte case to avoid calls
+ to mbrtowc and memmove/memcpy
+
+lib/readline/mbutil.c
+ - _rl_char_value: if we are in a UTF-8 locale (_rl_utf8locale) and the
+ current character does not have the eighth bit set ((c & 0x80) == 0),
+ return that char without bothering to call mbrtowc
+ - _rl_adjust_point: don't bother calling mbrlen if we are in a UTF-8
+ locale and the current character is not a multibyte character
+ - _rl_find_next_mbchar_internal: if we are in a UTF-8 locale, use that
+ to avoid calls to mbrtowc
+
+lib/readline/display.c
+ - _rl_col_width: if in a UTF-8 locale, take advantage of that to avoid
+ calls to mbrlen and mbrtowc
+ - rl_redisplay: if in a UTF-8 locale, take advantage of that to avoid
+ calls to mbrtowc
+
+ 11/17
+ -----
+lib/readline/histlib.h
+ - PATTERN_SEARCH: new flag for history search interfaces, means to
+ search for a pattern as Posix specifies
+
+lib/readline/histsearch.c
+ - history_search_internal: now takes a `flags' argument instead of just
+ an `anchored' argument to accommodate PATTERN_SEARCH
+ - _hs_history_patsearch: new internal interface to use when flags
+ specify PATTERN_SEARCH
+
+lib/readline/rlprivate.h
+ - SF_PATTERN: new flag for non-incremental search contexts, means to
+ treat search string as a pattern (maps to PATTERN_SEARCH)
+
+lib/readline/search.c
+ - noninc_dosearch: takes an additional `flags' argument for SF_PATTERN
+ - noninc_search_from_pos: takes an additional `flags' argument for
+ SF_PATTERN
+ - noninc_dosearch: pass flags argument through to noninc_search_from_pos
+ - _rl_nsearch_init: if we are in vi command mode, and `pchar' (key used
+ to invoke the search) is `?' or `/', add SF_PATTERN to the flags
+ to conform to Posix spec for vi-mode editing
+ - _rl_nsearch_dosearch: if cxt->sflags indicates that we should do a
+ pattern search, pass SF_PATTERN to noninc_search_from_pos
+ - rl_noninc_forward_search_again: if in vi command mode and key == `N',
+ make sure flags arg passed to noninc_dosearch includes SF_PATTERN
+ - rl_noninc_reverse_search_again: if in vi command mode and key == `n',
+ make sure flags arg passed to noninc_dosearch includes SF_PATTERN
+ - rl_history_search_internal: pass extra flags arg to noninc_search_from_pos
+
+ 11/18
+ -----
+lib/readline/search.c
+ - noninc_search_from_pos: takes a new argument, a pointer to int where
+ it stores the offset of the match in the history line (if non-NULL),
+ modified callers
+ - rl_history_search_internal: pass newcol pointer to
+ noninc_search_from_pos to avoid having to recompute the match offset
+ in the matching history entry
+
+config.h.in
+ - define HAVE_FNMATCH if we have the fnmatch function (configure was
+ already checking for it)
+
+lib/readline/histsearch.c
+ - _hs_history_patsearch: transform the pattern into one ending in a
+ `*' to avoid having to perform multiple match attempts from each
+ index in the history line
+ - history_search_internal: changes to handle pattern matching if
+ PATTERN_SEARCH is in the flags and HAVE_FNMATCH is defined. This
+ makes vi-mode history searching treat the user-specified search
+ string as an fnmatch pattern, as Posix requires. Fixes issue
+ reported by <jwdevel@gmail.com>
+
+ 11/19
+ -----
+builtins/shopt.def
+ - shopt_enable_hostname_completion: only declare this if READLINE
+ defined, since that's the only time it's used. Reported by
+ isabella parakiss <izaberina@gmail.com>
+
+ 11/20
+ -----
+builtins/common.c
+ - display_signal_list: make `kill -l 0' print `EXIT', modifying change
+ from 9/17
+
+ 11/21
+ -----
+subst.c
+ - expand_word_internal: when the shell encounters a backslash as the
+ last character of the string, don't try to add a '\'0' to the end
+ of the string if the string is quoted, just add a <CTLESC><\> and
+ skip to the end of string processing. Fixes oob-read error
+ reported by Jerzy Kramarz <op7ica@gmail.com>
+
+test.c
+ - two_arguments: check argv[pos][1] before checking argv[pos][2] when
+ looking for a unary argument. Fixes oob-read error on single `-'
+ in unary operator position reported by Jerzy Kramarz
+ <op7ica@gmail.com>
+ - ANDOR: check s[1] before checking s[2] in case s[1] == end of string.
+ Fixes oob-read error reported by Jerzy Kramarz <op7ica@gmail.com>
+
+subst.c
+ - command_substitute: now takes additional flags argument, changed one
+ caller (param_expand) to pass through PF_ASSIGNRHS flag; other callers
+ pass 0
+ - command_substitute: pass through flags argument to read_comsub
+ - read_comsub: now takes additional new flags argument to indicate
+ whether or not the word is in a context where word splitting will
+ eventually be performed
+ - read_comsub: if we are running in a context where word splitting
+ will not take place (PF_ASSIGNRHS), we read a CTLESC, and CTLESC is
+ in $IFS, add a CTLESC to make sure it gets through a round of
+ dequoting
+
+ 11/23
+ -----
+parse.y
+ - xparse_dolparen: since reset_parser frees the pushed string list,
+ and this is supposed to be a separate parsing context, we need to
+ save and restore pushed_string_list around the call to parse_string
+ and reset_parser. Fixes bug reported by Dan Douglas
+ <ormaaj@gmail.com>
+
+bashhist.h
+ - HISTEXPAND_DEFAULT: default value of history_expansion, moved here
+ from flags.c
+
+flags.[ch]
+ - histexp_flag: new variable, this is what set -H/set -o histexpand
+ sets; history_expansion set to value of this variable; defaults to 0
+ - history_expansion: now set to HISTEXPAND_DEFAULT
+
+bashhist.c
+ - bash_history_reinit: if interact == 0, set history_expansion to the
+ value of histexp_flag, to allow -H option to enable history
+ expansion
+ - bash_history_reinit: if interact == 0, make sure
+ history_expansion_inhibited is set opposite of histexp_flag, so
+ enabling history expansion with -H invocation option turns off
+ inhibited expansion. If interactive shell, we turn it off so
+ history_expansion controls whether or not history expansion takes
+ place. In practice, this function is always called with interact == 0,
+ and -c command still doesn't perform history expansion
+
+shell.c
+ - init_interactive: make sure histexp_flag and history_expansion are
+ identical in an interactive shell; allows both -H/+H option and
+ default compilation options (HISTEXPAND_DEFAULT) to work
+
+ 11/24
+ -----
+subst.c
+ - get_word_from_string: take advantage of the fact that SEPARATORS is
+ always a (possibly local) copy of IFS, so build a local charmap of
+ separators and use it instead of the (cached and possibly stale)
+ ifs_cmap map and isifs macro. Fixes bug with -N stripping leading
+ whitespace reported by Clark Wang <clarkw@vmware.com>
+
+ 11/25
+ -----
+builtins/read.def
+ - read_builtin: make `i' volatile, since auto variables are technically
+ undefined after a longjmp, and a timeout on SIGALRM causes one.
+ Fix suggested by Dmitry Goncharov <dgoncharov@users.sf.net>
+
+ 11/27
+ -----
+parse.y
+ - shell_getc: if we are reading input from a string, and we get the
+ ending '\0', and we are not expanding an alias, return EOF right
+ away. It might be an eval'ed string that has a syntax error.
+ Fixes bug reported by Dan Douglas <ormaaj@gmail.com>
+
+ 11/28
+ -----
+subst.c
+ - expand_word_internal: if we're expanding a quoted string containing
+ a tilde as the first character of a word, only expand the tilde if
+ we're expanding an array subscript (Q_ARRAYSUB), not if we're
+ expanding any arithmetic expression (Q_ARITH). This fixes a
+ backwards compatibility issue with expressions like ~0 reported by
+ Bize Ma <binaryzebra@gmail.com>
+
+subst.c
+ - parameter_brace_expand_indir: if the variable is not special and
+ the indirect variable is unset (!variable where variable is unset),
+ report an error. This is a slight incompatibility with previous
+ versions, which did not differentiate between indirecting an unset
+ variable and an indirect expansion resulting in an unset variable.
+ Fixes omission reported by otenba@protonmail.com
+
+ 11/30
+ -----
+lib/readline/doc/{rluser.texi,readline.3},doc/bash.1
+ - do-uppercase-version: since this doesn't actually exist, replace it
+ with do-lowercase-version, with the caveat that do-lowercase-version
+ results in undefined behavior if bound to a character that's
+ already lowercase. Suggested by Branden Robinson
+ <g.branden.robinson@gmail.com>
+
+lib/readline/doc/rluser.texi
+ - history-substring-search-forward: was misspelled as "substr-search";
+ ditto for search-backward
+
+doc/bash.1
+ - history-substring-search-{backward,forward}: add documentation, text
+ straight from readline.3
+
+jobs.c
+ - initialize_job_control: if the terminal's process group is valid,
+ but the shell is not the foreground process, check for terminating
+ signals (like SIGHUP, which the kernel would send if the process
+ group were orphaned) during the loop where we try to stop the shell
+ if it's not in the foreground. Not a perfect solution; the kernel
+ might not send SIGHUP
+
+ 12/1
+ ----
+jobs.c
+ - initialize_job_control: only go through the SIGTTIN loop 16 times at
+ most before deciding we're in the background and probably a member of
+ an orphaned process group; set original_pgrp to the terminal's pgrp
+ (so the eventual give_terminal_to doesn't leave the terminal in the
+ wrong pgrp); and bail on job control initialization
+
+tests/run-all
+ - unset SHELLOPTS to avoid its options `polluting' the test
+ environment. Suggestion from Vladimir Marek
+ <Vladimir.Marek@oracle.com>
+
+tests/execscript
+ - use $PWD/exec8.sub to protect against someone who has a cd command
+ in his bashrc. Report from Vladimir Marek
+ <Vladimir.Marek@oracle.com>
+
+ 12/2
+ ----
+shell.c
+ - open_shell_script: call end_job_control before exiting the shell in
+ case we were run as `bash -i script' and there is a problem with
+ executing `script'. If we don't, the terminal ends up in the wrong
+ process group. Fix from Clark Wang <clarkw@vmware.com>
+
+ 12/3
+ ----
+parse.y
+ - read_comsub: if we look for a character following a `<' and it's
+ not `<' (indicating that we don't have to read a here-document),
+ make sure we push that character back and go through the loop again
+ so we note the right word start location. Fixes bug reported by
+ parasite parasite <42parasite@gmail.com>
+
+subst.c
+ - parameter_brace_expand_indir: clean up checks for unset variables to
+ minimize incompatibilities with previous versions
+
+ 12/5
+ ----
+lib/readline/history.c
+ - add_history: don't update history_length until all of the updates to
+ the_history are complete, so history_length is always consistent.
+ Report and fix from Hong Cho <hong.cho@citrix.com>
+
+bashline.c
+ - bash_backward_shellword: fix to work better at the end of the line,
+ with multiple trailing whitespace characters. Inspired by report
+ from Kirill Timofeev <kt97679@gmail.com>
+
+Makefile.in
+ - bashbug: use `^' as the sed delimiter instead of `%'; some compilers
+ use `%' in CFLAGS. Report from Vladimir Marek
+ <Vladimir.Marek@oracle.com>
+
+ 12/6
+ ----
+test.c
+ - test_unop: make sure op[1] != 0 before testing op[2]
+ - term: make sure argv[pos][1] != 0 before testing argv[pos][2]. Fixes
+ oob-read reported by Jerzy Kramarz <op7ica@gmail.com>
+
+ 12/7
+ ----
+lib/glob/glob.c
+ - glob_filename: assign the return value of realloc (results, ...) to
+ a temp pointer, so if realloc returns NULL, we make sure we free
+ results. Report from Kirill Timofeev <kt97679@gmail.com>
+
+ 12/12
+ -----
+lib/sh/shquote.c
+ - sh_backslash_quote: if FLAGS&2, backslash-quote other single-byte
+ characters that are in the current locale's <blank> character class
+
+builtins/printf.def
+ - printf_builtin: when processing the `%q' format, make sure to
+ backslash quote all <blank> characters by making sure that FLAGS
+ passed to sh_backslash_quote includes 2. Fixes issue for certain
+ iso-8859-1 locales where character 160 (octal 240, hex 0xa0) is a
+ non-breaking space. Report from Stephane Chazelas
+ <stephane.chazelas@gmail.com>
+
+lib/sh/eaccess.c
+ - sh_stat: if DEV_FD_STAT_BROKEN is defined, take /dev/fd/N and turn
+ it into a stat on file descriptor N, as if /dev/fd were not
+ available. This is the case on some old versions of SunOS. Report
+ and patch from Dmitry Goncharov <dgoncharov@users.sf.net>
+
+config.h.in
+ - DEV_FD_STAT_BROKEN: add define
+
+configure.ac
+ - --enable-dev-fd-stat-broken: new command-line option, defines
+ DEV_FD_STAT_BROKEN if supplied at configure time
+
+doc/bashref.texi
+ - --enable-dev-fd-stat-broken: document new command-line option for
+ configure
+
+arrayfunc.c
+ - assoc_expand_once: new variable, declared here just because
+
+arrayfunc.h
+ - assoc_expand_once: extern declaration
+ - AV_NOEXPAND: new flag value for the array_value family of functions:
+ means to not run associative array subscripts through word expansion
+
+builtins/set.def
+ - unset_builtin: call unbind_array_element with assoc_expand_once as
+ third arg, controls whether we expand associative array subscripts
+ (if 1, we suppress the usual expansion). With accompanying shopt,
+ provides backwards compatible option to solve problem with quotes
+ and other characters in associative array subscripts pointed out
+ most recently by Mingye Wang (Arthur2e5) <arthur200126@gmail.com>
+
+builtins/shopt.def
+ - assoc_expand_once: new option, controls value of internal variable
+ with the same name. Currently undocumented because I'm not completely
+ sold on the name
+
+expr.c
+ - evalexp: now takes a second argument, flags. Changed all callers in
+ multiple other files (builtins/let.def,execute_cmd.c,arrayfunc.c,
+ subst.c,test.c,variables.c)
+
+externs.h
+ - EXP_EXPANDED, new flag for second argument to evalexp
+
+builtins/let.def
+ - let_builtin: add EXP_EXPANDED to flags passed to evalexp
+
+execute_cmd.c
+ - execute_arith_command: add EXP_EXPANDED to flags passed to evalexp,
+ since the command string is run through expand_words_no_vars before
+ being evaluated
+
+expr.c
+ - set expression-global variable already_expanded if flags arg to
+ evalexp contains EXP_EXPANDED
+ - if assoc_expand_once option set and already_expanded flag set, pass
+ 1 in flags to array_variable_part to skip over quotes and expansions
+ in the subscript
+ - if those two variables set, pass AV_NOEXPAND as flag to
+ get_array_value to prevent word expansions on the subscript
+
+arrayfunc.c
+ - if flags passed to unbind_array_element includes 1, pass flag on to
+ skipsubscript so we don't expect matched quotes or process
+ expansions in the subscript
+ - if flags to array_value_internal include AV_NOEXPAND, pass 1 as flag
+ to array_variable_part to pass along to skipsubscript
+ - if flags to array_value_internal include AV_NOEXPAND, don't call
+ expand_assignment_string_to_string; just use the unexpanded subscript
+ to produce the key [THIS IS A WORK IN PROGRESS]
+
+ 12/14
+ -----
+subst.h
+ - ASS_NOEXPAND: assignment flag that inhibits expansion of associative
+ array subscripts
+
+variables.c
+ - bind_int_variable: takes new flags arg; flags are taken from the
+ ASS_ set of assignment flags, changed callers (bashline.c, expr.c,
+ pcomplete.c)
+ - bind_int_variable: if flags includes ASS_NOEXPAND, pass 1 as flag to
+ valid_array_reference and array_variable_part to pass along to
+ skipsubscript, so we don't try to skip over quoted strings in the
+ subscript
+ - bind_int_variable: pass flags along to assign_array_element
+
+variables.h
+ - bind_int_variable: updated extern declaration with new flags arg
+
+expr.c
+ - expr_bind_variable: if the assoc_expand_once option is enabled, and
+ the flags to evalexp indicate that we have already run the expression
+ through word expansion, pass ASS_NOEXPAND as flag to bind_int_variable
+
+arrayfunc.c
+ - assign_array_element: if flags includes ASS_NOEXPAND, pass 1 as flag
+ to array_variable_name to pass along to skipsubscript
+ - assign_array_element: if flags includes ASS_NOEXPAND, don't run an
+ associative array subscript through word expansion, just use as-is
+
+subst.c
+ - param_expand: call evalexp with EXP_EXPANDED flag for arithmetic
+ substitution because the string has already been expanded with
+ expand_arith_string
+
+ 12/15
+ -----
+builtins/read.def
+ - read_builtin: use value of assoc_expand_once for valid_array_reference
+ as with other uses
+ - bind_read_variable: if assoc_expand_once is set, pass ASS_NOEXPAND
+ to assign_array_element
+
+general.c
+ - assignment: instead of checking whether flags == 0 to allow a `[',
+ explicitly check for (flags&1) to disallow it. This leaves the door
+ open for additional flag values
+
+builtins/printf.def
+ - printf_builtin: use value of assoc_expand_once for
+ valid_array_reference as with other uses
+ - bind_printf_variable: if assoc_expand_once is set, pass ASS_NOEXPAND
+ to assign_array_element
+
+ 12/16
+ -----
+builtins/history.def
+ - change history -d option to handle negative arguments; negative
+ arguments offset from the end of the history list (last_position + 1
+ so history -d -1 deletes the history -d command that just got
+ added). Original patch from Piotr Grzybowski <narsil.pl@gmail.com>
+
+doc/bash.1,lib/readline/doc/hsuser.texi
+ - documented new behavior of negative offsets for `history -d'
+
+ 12/17
+ -----
+lib/readline/history.c
+ - remove_history: use memmove to move the history list around instead
+ of a loop that copies pointers one at a time, similar to add_history
+
+ 12/19
+ -----
+configure.ac
+ - linux: make sure PGRP_PIPE is defined unconditionally for all kernel
+ versions greater than 2.3.*
+
+lib/readline/history.c
+ - clear_history: when clearing the history, reset history_base to 1
+ (its default)
+ - remove_history_range: new function, removes a set of history entries
+ and returns them all for deallocation
+
+lib/readline/history.h
+ - remove_history_range: extern declaration
+
+ 12/20
+ -----
+execute_cmd.c
+ - eval_arith_for_expr: pass EXP_EXPANDED to evalexp, since we run the
+ expression through expand_words_no_vars
+
+ 12/21
+ -----
+input.c
+ - b_fill_buffer: when read returns EOF or error, and we reset the
+ buffer and pointers, reset the input pointer to 0. Fixes issue
+ reported by Stephane Chazelas <stephane.chazelas@gmail.com>
+
+ 12/22
+ -----
+arrayfunc.[ch],subst.c
+ - array_expand_index: added new FLAGS argument, reserved for future use,
+ changed callers
+
+
+ 12/24
+ -----
+variables.c
+ - can_optimize_assignment,optimized_assignment: see if we can detect
+ whether or not we're trying to append a string to a string variable,
+ and optimize that using realloc/strcpy. Based on a log-ago
+ suggestion from Aharon Robbins <arnold@skeeve.com>
+
+ 12/27
+ -----
+builtins/help.def
+ - help_builtin: don't pass return value from ngettext() directly to
+ printf
+
+ 12/28
+ -----
+sig.c
+ - set_signal_handler: set SA_RESTART for SIGWINCH, so window size
+ changes don't interrupt open/read/write. Suggested by
+ Andriy Prystupa <andriy.prystupa@globallogic.com> back in 10/2016
+
+ 12/30
+ -----
+execute_cmd.c
+ - coproc_pidchk: if MULTIPLE_COPROCS, call cpl_search to find coproc
+ associated with pid, don't remove it from the list with cpl_delete.
+ Just mark as dead and wait for coproc_reap/cpl_reap to take care
+ of disposing it
+ - execute_coproc: NEW FEATURE: run the coproc name through word
+ expansion before creating the coproc; this allows unique coprocs
+ to be created in loops. Originally requested way back in 2012 by
+ Erik Brandsberg <ebrandsberg@juniper.net>
+
+command.h
+ - subshell_com: now has a line number member: line
+
+make_cmd.c
+ - make_subshell_command: assign current line number (line number of the
+ ending right paren) to `line' member
+
+copy_cmd.c
+ - copy_subshell_command: copy the `line' member
+
+execute_command.c
+ - execute_command_internal: when executing a user subshell (`(command)'),
+ set the line number and line number for the ERR trap from the line
+ number saved in the subshell command struct. Fix from
+ Eduardo A. Bustamante López <dualbus@gmail.com>
+ - execute_arith_command: set line number for ERR trap from line number
+ saved in the arith command struct. Fix from
+ Eduardo A. Bustamante López <dualbus@gmail.com>
+ - execute_cond_command: set line number for ERR trap from line number
+ saved in the cond command struct. Fix from
+ Eduardo A. Bustamante López <dualbus@gmail.com>
+
+variables.c
+ - bind_variable_internal: move can_optimize_assignment call to the
+ right place
+
+ 12/31
+ -----
+variables.c
+ - find_variable_nameref: treat a circular variable reference as
+ referring to a variable of that name in the global scope (for now),
+ if one exists. PROVISIONAL change: inspired by suggestion back in
+ 6/2016 by Grisha Levit <grishalevit@gmail.com>
+
+ 1/2/2017
+ --------
+{expr,general,hashcmd}.c
+ - include flags.h for flag values
+
+make_cmd.c
+ - include shell.h instead of the files it includes
+
+flags.h
+ - new extern declarations for want_pending_command, read_from_stdin
+
+subst.h
+ - add extern declaration for expanding_redir, assigning_in_environment
+
+trap.h
+ - add extern declaration for running_trap
+
+execute_cmd.h
+ - added extern declarations for variables declared in execute_cmd.c,
+ changed files that declare lots of extern variables
+
+shell.h
+ - added extern declarations for variables declared in shell.c,
+ changed files that declare lots of extern variables
+
+parser.h
+ - added extern declarations for variables declared in parse.y,
+ changed files that declare lots of extern variables
+
+[lots of changes to reduce the number of extern declarations in C and DEF files]
+
+parse.y
+ - xparse_dolparen: need ALIAS or DPAREN_ARITHMETIC to be defined to
+ use STRING_SAVER
+
+ 1/3
+ ---
+sig.c
+ - termsig_handler: don't call end_job_control if subshell_environment
+ indicates that we're in a command or process substitution.
+ Provisional fix for bug reported by Russell King <rmk@armlinux.org.uk>
+
+jobs.c
+ - end_job_control: only attempt to give the terminal back to
+ original_pgrp if job_control != 0; don't test interactive_shell
+
+execute_cmd.c
+ - execute_disk_command: call QUIT after search_for_command, since
+ the only time QUIT gets called is when the path is actually
+ searched. It doesn't happen for an absolute pathname. Fix for
+ bug reported by Russell King <rmk@armlinux.org.uk>
+
+ 1/16
+ ----
+lib/glob/glob.c
+ - extglob_skipname,wextglob_skipname: if we don't find a reason to
+ not skip the name in any subpattern (r returns 0 for any subpattern),
+ return 1 because we should skip the name
+ - extglob_skipname,wextglob_skipname: if we don't find a reason to
+ not skip the name in any subpattern, but the first part of the extglob
+ pattern is a wildcard (`*(pat)'), check the rest of the pattern, if
+ any, to decide whether or not to skip the name. Fixes bug reported
+ by Grisha Levit <grishalevit@gmail.com>
+
+lib/glob/glob_loop.c
+ - CHAR -> GCHAR (unsigned char); change uses of CHAR to GCHAR
+
+lib/glob/glob.c
+ - wchkname -> wskipname
+ - wskipname: now takes a third `flags' argument, like the other variants
+ of skipname
+
+ 1/17
+ ----
+redir.c
+ - here_document_to_fd: set the file descriptor open for writing to the
+ here document to be close-on-exec. Fixes bug reported by
+ Siteshwar Vashisht <svashisht@redhat.com>
+
+ 1/18
+ ----
+{gmisc,gm_loop}.c
+ - match_pattern_char,match_pattern_wchar: moved to gm_loop.c (new file),
+ compile twice for single and wide characters as with the rest of
+ the functions
+ - umatchlen,wmatchlen: ditto
+ - extglob_pattern_p: move to gm_loop.c, generalize for regular and
+ wide-char versions
+
+ 1/19
+ ----
+lib/readline/complete.c
+ - stat_char: fix small memory leak if stat/lstat returns -1
+
+bashline.c
+ - bash_filename_stat_hook,bash_directory_completion_hook: make sure
+ expand_prompt_string is caled wth W_NOPROCSUB flag
+ - directory_exists: don't dequote the filename passed as an argument;
+ this function should never get quoted pathnames
+
+subst.c
+ - expand_word_internal: when calling expand_word_internal recursively
+ after reading a double-quoted string, make sure the W_NOCOMSUB and
+ W_NOPROCSUB flags make it to the recursive invocation. Fixes bug
+ reported by Jens Heyens <jens.heyens@cispa.saarland>
+
+ 1/23
+ ----
+lib/readline/signals.c
+ - _rl_orig_sigset: original signal mask, set and restored by
+ rl_set_signals (rl_clear_signals doesn't block signals). If we
+ are not installing signal handlers, just save signal mask each
+ time rl_set_signals is called
+
+lib/readline/input.c
+ - rl_getc: use _rl_orig_sigmask in the call to pselect(), so we block
+ the set of signals originally blocked by the calling application.
+ Fixes bug reported by Frédéric Brière <fbriere@fbriere.net>
+
+parse.y
+ - yy_readline_get: try to unset NONBLOCK mode on readline's input
+ file descriptor before calling readline(). Inspired by report from
+ Siteshwar Vashisht <svashisht@redhat.com>
+
+ 1/30
+ ----
+parse.y
+ - parse_comsub: don't unconditionally set the LEX_WASDOL flag when we
+ see a `$'; we should toggle it to accommodate things like $$.
+ Fixes bug reported by Christian Weisgerber <naddy@mips.inka.de>
+ - parse_matched_pair: do the same thing with LEX_WASDOL
+
+ 2/1
+ ---
+jobs.h
+ - JWAIT_PERROR,JWAIT_FORCE: new defines for the wait_for_ family of
+ functions
+
+jobs.c
+ - wait_for_job, wait_for_any_job: take new flags argument for use by
+ the wait builtin
+ - wait_for_job: if JWAIT_FORCE flag supplied, loop waiting for job to
+ terminate instead of change state
+ - wait_for_single_pid: if JWAIT_FORCE flag supplied, loop waiting for
+ child process to terminate instead of change state
+
+builtins/wait.def
+ - takes a new `-f' option, signifying to wait until the specified job
+ or process terminates, instead of changes state
+
+doc/{bash.1,bashref.texi}
+ - wait: document the new -f option and its behavior when job control
+ is enabled
+ - job control: add a paragraph describing how the wait builtin changes
+ behavior when job control is enabled: it returns when a job changes
+ state
+
+ 2/2
+ ---
+examples/loadables/fdflags.c
+ - fdflags: new loadable builtin to set and unset file descriptor flags
+ (such as non-blocking, though bash undoes that) for descriptors bash
+ has open. Developed in consultation with Christos Zoulas
+ <christos@zoulas.com>
+
+ 2/3
+ ---
+
+lib/readline/text.c
+ - rl_previous_screen_line: attempt to move to the same column on the
+ previous screen line of the current readline buffer by moving back
+ a number of characters equal to the screen width
+ - rl_next_screen_line: attempt to move to the same column on the
+ next screen line of the current readline buffer by moving forward
+ a number of characters equal to the screen width. Both originally
+ suggested by Hans Ginzel <hans@matfyz.cz>
+
+lib/readline/readline.h
+ - rl_{next,previous}_screen_line: extern declarations
+
+lib/readline/funmap.c
+ - {next,previous}-screen-line: new bindable command names
+
+lib/readline/doc/{rluser.texi,readline.3}
+ - {next,previous}-screen-line: document behavior, including limitations
+
+ 2/5
+ ---
+lib/readline/input.c
+ - rl_read_key: when reading input from a macro, make sure to return
+ the next character as an unsigned char, so we handle multibyte
+ character sequences correctly. Report and fix from Grisha Levit
+ <grishalevit@gmail.com>
+
+ 2/6
+ ---
+shell.c
+ - --pretty-print: new invocation option, currently undocumented, dumps
+ a pretty-printed version of a shell script given as an argument to
+ stdout
+ - main: call pretty_print_loop() if we're in pretty-printing mode in
+ a non-interactive shell
+
+eval.c
+ - pretty_print_loop: new function, similar to reader_loop, reads
+ commands and prints them by displaying the result from
+ make_command_string()
+
+externs.h
+ - pretty_print_loop: new extern declaration
+
+ 2/9
+ ---
+doc/{bash.1,bashref.texi}
+ - here strings: correct documentation to note they do not undergo
+ brace expansion. Report from Conor McCarthy <mr.spuratic@gmail.com>
+
+ 2/9
+ ---
+lib/readline/display.c
+ - expand_prompt: instead of recomputing the line break offsets for
+ prompts that span multiple screen lines on every call to rl_redisplay,
+ do it here. Manage a local array local_prompt_newlines; set an offset
+ every time `physchars' hits a multiple of the screen width. Inspired
+ by https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=843819
+ - rl_redisplay: don't calculate multi-line prompt line break indices
+ here; use the values calculated by expand_prompt
+ - rl_{save,restore}_prompt: save and restore local_prompt_newlines
+
+ 2/11
+ ----
+lib/readline/mbutil.c
+ - _rl_find_prev_mbchar_internal: since we're traversing the string
+ from the beginning, shortcut non-multibyte characters in a UTF-8
+ locale
+
+parse.y
+ - parse_comsub: if we are parsing a here document with a quoted
+ delimiter (LEX_QUOTEDDOC), we should not remove <backslash><newline>
+ pairs from the body of the here document. Need to add LEX_QUOTEDDOC
+ to the values in tflags when calling shell_getc. Fixes bug reported
+ by Michael Homer <michael.homer@ecs.vuw.ac.nz>, patch was originally
+ contributed by Geir Hauge <geir.hauge@gmail.com>
+
+ 2/15
+ ----
+jobs.c
+ - bgp_add: if hashed pid collides with bgpids.head index, print a
+ warning and move to the next index. Attempt to avoid collision
+ problem reported by Graham Northup <northug@clarkson.edu>
+
+ 2/17
+ ----
+subst.c
+ - process_substitute: call remove_quoted_escapes on the command string
+ like command substitution does, since the string will be run through
+ the parser again. Fixes bug reported by David Simmons
+ <bug-bash@tmp.davidsimmons.com>
+
+config-top.h
+ - STATIC_PATH_VALUE: new config variable, undefined by default
+
+variables.c
+ - shell_initialize: if STATIC_PATH_VALUE is defined, use it to set the
+ PATH variable at shell startup, overriding the environment and any
+ value for DEFAULT_PATH_VALUE. Based on a suggestion from
+ Lonnie Abelbeck <lonnie@abelbeck.com>
+
+ 2/18
+ ----
+shell.c
+ - maybe_make_restricted: clear the hash table before making the PATH
+ variable read-only by calling stupidly_hack_special_variables.
+ Suggestion from Lonnie Abelbeck <lonnie@abelbeck.com>
+
+subst.c
+ - expand_string_for_pat: change expansion of WORD in ${param[%[%]][#[#]]word}
+ to use this new function, initially identical to
+ expand_string_for_rhs
+ - getpattern: use expand_string_for_pat to expand the pattern
+
+ 2/19
+ ----
+subst.c
+ - expand_string_for_rhs: now that this is only used to expand the RHS
+ of the various non-pattern ${paramOPword} expansions, set
+ expand_no_split_dollar_star depending on whether or not the entire
+ brace expansion is going to be subjected to word splitting: if it's
+ quoted or if IFS is set but null, we will not be splitting so we set
+ this to 1 and let $* expand to separate fields separated by a space.
+ This was heavily debated back in October 2014 and resulted in Posix
+ interp 888. Reported by Martijn Dekker <martijn@inlv.org>
+
+ 2/20
+ ----
+subst.c
+ - expand_string_for_rhs: now takes `op' (brace expansion operator) as
+ an additional argument; changed call in parameter_brace_expand_rhs
+ - expand_string_for_rhs: make sure to set expand_no_split_dollar_star
+ if `op' is `=', since we are supposed to preserve the assignment
+ statement expansion semantics on the RHS of ${param=word}
+
+jobs.c
+ - bgp_delindex: make sure to set ps->pid to NO_PID so we more
+ effectively remove it from the hash table. It shouldn't matter as
+ long as nothing in pidstat_table points to it, but better to be safe
+ - bgp_delete,bgp_search: generalize previous collision detection to
+ loop detection: abort search if we are going to go back to the
+ first index we tried
+
+ 2/21
+ ----
+lib/readline/vi_mode.c
+ - rl_vi_bword: if we're trying to move backward to non-whitespace
+ in a line that begins with whitespace, make sure we don't move
+ point to 0 and test memory locations before the start of the
+ buffer. Report and fix from Jason Hood <jadoxa@yahoo.com.au>
+
+lib/readline/display.c
+ - expand_prompt: if _rl_screenwith == 0, call rl_reset_screen_size to
+ initialize it. If it's still 0 after that, guess that the screen
+ width is 80 and use that to guess how many screen lines the prompt
+ consumes
+
+lib/readline/readline.c
+ - readline_internal_setup: if in emacs editing mode, and we are
+ showing the editing mode in the prompt, call _rl_reset_prompt() to
+ make sure that the updated setting of _rl_show_mode_in_prompt
+ (presumably set in an inputrc file) is reflected in the redisplay
+ performed by rl_callback_newline via readline_internal_setup.
+ Fix from Jason Hood <jadoxa@yahoo.com.au>
+
+ 2/22
+ ----
+builtins/complete.def
+ - print_one_completion,print_compopts: make sure to display the
+ `nosort' and `noquote' options if they've been set for a particular
+ compspec. Fix from Grisha Levit <grishalevit@gmail.com>
+
+builtins/read.def
+ - read_builtin: if -e is supplied, read needs to free rlbuf before
+ returning, since it uses discard_unwind_frame at that point. Bug
+ and fix from ZhangXiao <xiao.zhang@windriver.com>
+
+execute_cmd.c
+ - execute_for_command: when using a nameref as the for loop control
+ variable, make sure we check for readonly namerefs and make sure
+ that we force bind_variable to enforce restrictions on nameref
+ assignments. Fix from Grisha Levit <grishalevit@gmail.com>
+
+ 2/23
+ ----
+general.c
+ - make_absolute: convert deprecated cygwin_conv_to_full_posix_path
+ to cygwin_conv_path (CCP_WIN_A_TO_POSIX, ...)
+
+{shell,variables}.c,config-top.h
+ - modify change from 2/17: rename STATIC_PATH_VALUE to RBASH_STATIC_PATH_VALUE;
+ don't use it in shell_initialize; set PATH to it just before
+ imposing restricted shell restrictions
+
+ 2/24
+ ----
+subst.c
+ - expand_string_for_rhs: revert change from 2/19 concerning setting
+ expand_no_split_dollar_star so we are consistent with the Posix
+ examples. This anticipates Martijn Dekker initiating an austin
+ group discussion about this case
+
+ 2/25
+ ----
+pcomplete.c
+ - gen_shell_function_matches: if COMPREPLY is an associative array,
+ don't coerce it to an indexed array, which will fail to produce any
+ matches anyway. Report from Grisha Levit <grishalevit@gmail.com>
+ - bind_comp_words: if COMP_WORDS exists as a nameref variable, ignore
+ the nameref attribute. Report from Grisha Levit
+ <grishalevit@gmail.com>
+
+subst.c
+ - parameter_brace_expand: if expand_no_split_dollar_star is set, make
+ sure chk_atstar treats ${var[*]} as quoted by passing Q_DOUBLE_QUOTES
+ as the second argument. Fixes differing treatment of $* and ${v[*]}
+ reported by Grisha Levit <grishalevit@gmail.com>, because otherwise
+ *contains_dollar_at gets set to 1 and the word is inappropriately
+ split
+
+ 3/1
+ ---
+subst.c
+ - expand_word_list_internal: if a variable assignment fails (e.g., due
+ to a readonly variable) in a non-interactive posix mode shell, don't
+ abort the shell if the assignment is being performed by the command
+ builtin, even if it's something like `command eval v=1'. Fixes
+ RedHat bug https://bugzilla.redhat.com/show_bug.cgi?id=1389838
+ (reported by Denys Vlasenko)
+
+ 3/3
+ ---
+builtins/shopt.def
+ - set_compatibility_level: make sure that changing the compatibility
+ level sets or modifies BASH_COMPAT. Fixes bug reported by
+ Stephane Chazelas <stephane.chazelas@gmail.com>
+
+lib/glob/smatch.c
+ - is_cclass: broke into two functions: is_valid_cclass and cclass_test,
+ for future expansion. No functionality change
+
+lib/glob/sm_loop.c
+ - BRACKMATCH: when testing a character against a character class name,
+ use the unfolded (original) character, since we're not performing a
+ comparison. This is what glibc does. Fixes bug reported by
+ Lukáš BaÅinka <lukas.barinka@boxtrap.net>
+
+ 3/4
+ ---
+doc/{bash.1,bashref.texi}
+ - note that the arguments to -eq and the rest of the arithmetic
+ operators are evaluated as expressions when using [[. Report from
+ Rob la Lau <rob@ohreally.nl>
+
+
+ 3/7
+ ---
+lib/readline/display.c
+ - update_line: completely rewrote the code that handles line wrapping
+ in the presence of multibyte/double-width characters; punts to a
+ dumb update in cases where the number of screen positions doesn't
+ match
+ - update_line: in one case, where we are deleting characters from the
+ line, if we are writing a non-space into the last screen character
+ position, note that we are no longer using a space as a placeholder
+ for a line wrap where there is a multi-width character that won't
+ fit on the screen line (wrapped_line[current_line+1] = 0)
+
+ 3/10
+ ----
+lib/readline/readline.c
+ - bind_arrow_keys_internal: add default key bindings for control-
+ arrow keys, suggested by Per Bothner <per@bothner.com>
+
+ 3/13
+ ----
+doc/{bash.1,bashref.texi}
+ - clarify the prompt strings the undergo expansion (PS0, PS1, PS2, and
+ PS4) before being displayed. Based on a report from Grisha Levit
+ <grishalevit@gmail.com>
+
+ 3/14
+ ----
+input.c
+ - bufstream_ungetc: return EOF if being asked to unget a character to
+ a NULL buffer
+ - buffered_getchar: check buffered_fd is valid and buffers[buffered_fd]
+ is non-NULL before trying to dereference. Fixes bug reported by
+ floyd <floyd@floyd.ch>
+
+execute_cmd.c
+ - execute_null_command: if we fork to execute the command, the subshell
+ is not interactive
+ - execute_null_command: for safety's sake, if a redirection attached
+ to a null command affects fd 0 or the fd the shell is reading for
+ input, and the redirection can potentially change or close that file
+ descriptor, force a fork so the redirection is done in a subshell
+
+ 3/15
+ ----
+lib/readline/display.c
+ - update_line: if we have to output some bytes, and we are outputting
+ the prompt string, make sure we correct _rl_last_c_pos if we are
+ at the wrap offset -- what if all of the characters in the prompt
+ are invisible, so the visible length is 0? Fixes display bug
+ reported by Grisha Levit <grishalevit@gmail.com>
+
+execute_cmd.c
+ - undo_partial_redirects: new function, performs redirections specified
+ in redirection_undo_list and disposes of them. Intended to be used
+ to unwind redirections performed before a redirection error
+ - dispose_partial_redirects: new function, disposes of
+ redirection_undo_list and nulls the list
+ - execute_command_internal: use undo_partial_redirects
+
+execute_cmd.h
+ - undo_partial_redirects: extern declaration
+ - dispose_partial_redirects: extern declaration
+
+subst.c
+ - exp_jump_to_top_level: if we are throwing an expansion error while
+ expanding a redirection (expanding_redir != 0), undo any undoable
+ redirections by calling undo_partial_redirects() so we don't leave
+ any in place
+
+variables.c
+ - initialize_shell_variables: if SHELLOPTS is in the environment,
+ unset the readonly attribute on any existing variable with that
+ name. It may have been created by processing a -o option on the
+ command line. Fixes issue reported by Stephane Chazelas
+ <stephane.chazelas@gmail.com>
+
+ 3/16
+ ----
+lib/readline/histfile.c
+ - read_history_range: short-circuit and do nothing if the history file
+ is of 0 size
+ - read_history_range: make sure the buffer is zero-terminated so the
+ timestamp tests work right for history files of 1 character. Report
+ and recommended fix from Denis Martinez <denis.martinez@gmx.com>
+
+lib/readline/complete.c
+ - compute_lcd_of_matches: when trying to compute the common prefix
+ of a set of matches while performing case-insensitive completion
+ matching, don't sort the match list unless rl_sort_completion_matches
+ is non-zero. Fixes problem reported by Roee Bar <roee@arista.com>
+
+ 3/18
+ ----
+subst.c
+ - extract_dollar_brace_string: handle process substitution embedded in
+ a ${...} expansion in the same way as command substitution. Reported
+ by D630 <d630@posteo.net>
+
+parse.y
+ - LEX_GTLT: new flag, set by parse_matched_pair when a `<' or '>' is
+ read
+ - parse_matched_pair: handle an embedded process substitution like an
+ embedded command substitution if we are parsing an array index or
+ a dollar-brace expansion. Reported by D630 <d630@posteo.net>
+
+ 3/21
+ ----
+subst.h
+ - ASS_NOEVAL, ASS_NOLONGJMP: new values for assignments, used by
+ make_variable_value
+
+variables.c
+ - make_variable_value: ASS_NOEVAL: if this is set, don't attempt
+ arithmetic evaluation or case modification, just produce the value
+ (still honor ASS_APPEND)
+ - make_variable_value: ASS_NOLONGJMP: don't longjmp back to top_level
+ if arithmetic evaluation fails due to a syntax error
+ - bind_variable_internal: make sure aflags is passed to all calls to
+ make_variable_value
+ - push_temp_var: add ASS_NOLONGJMP to the flags passed to
+ bind_variable_internal. Fixes bug reported by D630 <d630@posteo.net>
+
+bashline.c
+ - bash_execute_unix_command: don't bother to attempt optimization by
+ checking whether the call to parse_and_execute changes $READLINE_LINE;
+ it can be defeated by the new code that optimizes appends to string
+ variables. Bug reported by Grisha Levit <grishalevit@gmail.com>
+
+ 3/22
+ ----
+subst.c
+ - expand_string_for_rhs: re-apply change from 2/19 based on discussion
+ on the austin-group mailing list about how unquoted ${var-$*}
+ expands when IFS is null. Originally reported by Martijn Dekker
+ <martijn@inlv.org>
+
+ 3/25
+ ----
+parse.y
+ - read_token_word: if we read a WORD immediately after a CASE, FOR, or
+ SELECT, increment `expecting_in_token' to note that we're going to
+ be reading IN as the next non-newline token
+ - special_case_tokens: if we're expecting an IN (expecting_in_token > 0)
+ and we have an "in", return IN and decrement expecting_in_token. This
+ is a better implementation of Posix grammar rule 6 than the previous
+ code in that function (which is still there for posterity, but should
+ eventually be removed). Fixes bug reported by Torka Noda
+ <kanito.tasoga@gmail.com>
+
+subst.c
+ - string_list_pos_params: if the expansion is not quoted, and IFS is
+ null, return a string containing the quoted positional parameters
+ separated by spaces. Callers know what to do with the string
+ - parameter_brace_substring,parameter_brace_remove_pattern,
+ parameter_brace_transform,parameter_brace_patsub,parameter_brace_casemod:
+ for the VT_POSPARMS case, after calling the operator-specific
+ expansion function, which eventually calls string_list_pos_params,
+ leave the returned string alone, since it's already quoted properly
+ - parameter_brace_expand: after calling the above functions to perform
+ expansions, if the expansion is being performed on an unquoted $*
+ when IFS is null, set the W_SPLITSPACE flag in the return WORD_DESC
+ so that splitting on spaces is performed on the already-quoted word.
+ Fixes bug reported by Grisha Levit <grishalevit@gmail.com>
+
+ 3/28
+ ----
+subst.c
+ - process_substitute: change so the subshell doesn't inherit the 'v'
+ option, like command substitution now does (as of bash-4.4).
+ Suggested by Grisha Levit <grishalevit@gmail.com>
+
+ 3/30
+ ----
+subst.c
+ - parameter_brace_casemod: now takes a pflags parameter, like
+ parameter_brace_patsub; changed callers
+ - parameter_brace_transform: now takes a pflags parameter, like
+ parameter_brace_patsub; changed callers
+ - pos_params_pat_subst,pos_params_modcase: if mflags indicates we're
+ on the rhs of an assignment statement and not going to perform
+ word splitting, we're expanding $*, and $IFS is null, make sure
+ string_list_pos_params treats the expansion as double-quoted, so
+ the positional parameters will be concatenated. Fixes bug reported
+ by Grisha Levit <grishalevit@gmail.com>
+
+ 3/31
+ ----
+doc/{bash.1,bashref.texi}
+ - add notes to the description of pipelines emphasizing that each
+ element of a pipeline is executed in a subshell, referring to the
+ description of a command execution environment, and that the
+ lastpipe option can modify that behavior. From a discussion with
+ Jean Delvare <jdelvare@suse.de>
+
+ 4/1
+ ---
+subst.c
+ - expand_word_internal: set had_quoted_null to note for later if we
+ expand '', "", or "$x" when x is unset
+ - expand_word_internal: if we expand "$param" (or "${param}") and we
+ come back with a single word with the W_HAVEQUOTEDNULL flag set,
+ make sure we set had_quoted_null to remember it for later. In this
+ case, if we already have seen a quoted null and we expanded $@ in
+ this (sub)string expansion, we don't want to do any of the special
+ $@ handling. In particular, we don't want to set *expanded_something,
+ because that will result in word splitting and quoted null removal.
+ Fixes bug reported by Grisha Levit <grishalevit@gmail.com>
+ - parameter_brace_patsub: when expanding ${*/pat/sub} in the case
+ where ifs_is_null != 0 and (pflags & PF_NOSPLIT2) (meaning we aren't
+ going to be doing word splitting), pass MATCH_ASSIGNRHS so we do
+ the right thing in this case. Part of a set of cases inspired by
+ Grisha Levit <grishalevit@gmail.com>
+ - param_expand: case '*': if we are on the rhs of an assignment
+ (PF_ASSIGNRHS) and ifs is unset, and $1 == " ", we need to make sure
+ string_list_dollar_at gets called with Q_DOUBLE_QUOTES so it quotes
+ the positional parameters before joining them into a string with
+ string_list_internal. Otherwise, that first space gets removed by
+ the implicit word splitting that gets performed. Reported by
+ Grisha Levit <grishalevit@gmail.com>
+
+array.c
+ - array_patsub,array_modcase: handle ${A[*]} identically to $* when
+ not quoted and IFS is null (separate with spaces). Makes things
+ like ${A[*]/x/y} behave the same as ${*/x/y} when IFS is null
+
+subst.c
+ - parameter_brace_patsub,parameter_brace_casemod: if expanding ${A[*]}
+ when the match is on the rhs of an assignment statement (PF_ASSIGNRHS)
+ and ifs is null, make sure the match is performed as if it were
+ quoted
+ - parameter_brace_substring: now takes a PFLAGS argument
+ - parameter_brace_substring: if expanding $* in a context where we
+ don't want to do word splitting (expand_no_split_dollar_star &&
+ PF_NOSPLIT2), make sure we expand $* as if it were quoted
+ - parameter_brace_substring: if expanding an unquoted $* on the rhs of
+ an assignment statement when IFS is null, make sure we expand the $*
+ as if it were quoted
+
+ 4/4
+ ---
+doc/{bash.1,bashref.texi}
+ - read: document that a delim argument of '' terminates each line when
+ a NUL character appears
+
+ 4/5
+ ---
+subst.c
+ - parameter_brace_expand_word: handle case where unquoted $@ is
+ expanded in a context where word splitting is not performed (PF_NOSPLIT2)
+ and IFS is not equal to the standard value
+ - param_expand: handle case where unquoted $@ is expanded in a context
+ where word splitting is not performed (PF_NOSPLIT2) and IFS is not
+ equal to the standard value. Fixes bug with IFS == $1 when expanding
+ unquoted $@ reported by Grisha Levit <grishalevit@gmail.com>
+
+ 4/7
+ ---
+subst.c
+ - pos_params_pat_subst: if MATCH_ASSIGNRHS is set, don't expand $*
+ as quoted unless expand_no_split_dollar_star is also set. Fixes
+ expansion bug reported on 4/4 by Grisha Levit <grishalevit@gmail.com>
+ - parameter_brace_transform: if we transform unquoted A[@] or A[*]
+ when IFS is null, don't quote the result of array_transform in the
+ same way we don't quote the result of parameter_list_transform,
+ since it will come back quoted. Fixes bug reported by
+ Grisha Levit <grishalevit@gmail.com>
+ - list_transform: if we transform unquoted $* or ${a[*]} when IFS is
+ null and expand_no_split_dollar_star is non-zero, call
+ string_list_pos_params with Q_DOUBLE_QUOTES to inhibit splitting.
+ We do this in all cases except @Q, which is inconsistent but more
+ intuitive.
+ Fixes bug reported by Grisha Levit <grishalevit@gmail.com>
+
+ 4/8
+ ---
+builtins/read.def
+ - read_mbchar: if unbuffered == 2, we are running read -N and need to
+ use zreadn to read the rest of the multibyte char, since the
+ previous call to zreadn may have buffered characters. Fixes bug
+ reported by Stephane Chazelas <stephane.chazelas@gmail.com>
+
+pathexp.h
+ - QGLOB_CTLESC: new flag for quote_string_for_globbing: means to turn
+ CTLESC CTLESC (quoted CTLESC that's usually removed as part of
+ word splitting or quote removal) into CTLESC
+
+pathexp.c
+ - quote_string_for_globbing: if QGLOB_CTLESC is present in flags,
+ turn CTLESC CTLESC into CTLESC, even if the first CTLESC is preceded
+ by a backslash. Callers can't do this because the remaining CTLESC
+ will be treated as a quoting character and turned into a backslash
+
+execute_cmd.c
+ - execute_case_command: if the pattern is not quoted, make sure to tell
+ quote_string_for_globbing to turn CTLESC CTLESC into CTLESC by
+ passing QGLOB_CTLESC. Fixes bug reported 4/2 by Martijn Dekker
+ <martijn@inlv.org>
+
+ 4/9
+ ---
+subst.c
+ - string_transform: after calling ansiexpand for the `E' transformation,
+ need to call dequote_escapes because ansiexpand assumes it is being
+ called by the parser and arranges for ansicstr to quote CTLESC and
+ CTLNUL with CTLESC. Reported by Grisha Levit <grishalevit@gmail.com>
+
+variables.c
+ - mk_env_string: when exporting functions, run them through
+ dequote_escapes because any strings containing CTLESC or CTLNUL are
+ still quoted with CTLESC. Reported by Grisha Levit
+ <grishalevit@gmail.com>
+
+subst.[ch]
+ - quote_escapes,dequote_escapes: now take const char * argument
+
+execute_cmd.c
+ - execute_intern_function: if the function name contains CTLESC, run
+ it through dequote_escapes just in case it got CTLESC quoting
+ CTLESC or CTLNUL from the parser
+
+ 4/11
+ ----
+eval.c
+ - reader_loop: make sure PS0 is expanded and displayed before
+ incrementing the command number
+
+parse.y
+ - prompt_history_number: function to make history number as reported
+ by \!, or posix mode !!, be the same index that the (possibly multi-
+ line) command will be stored with. The history library increments
+ history_offset as soon as a new line is entered, and it's up to the
+ history code to store the second and subsequent lines of a command
+ in the same history entry. If we are expanding PS0 or PS4, or
+ a ${var@P} expansion, we're already past the history entry no
+ matter what. Inspired by patch from Grisha Levit
+ <grishalevit@gmail.com>
+ - decode_prompt_string: call prompt_history_number
+ - decode_prompt_string: if expanding PS4 or ${var@P}, make the
+ command number the same as reported by PS0/PS1/PS2
+
+ 4/12
+ ----
+support/Makefile.in
+ - clean: remove man2html.o as part of this production instead of
+ waiting until `distclean'. Reported by chrlis <chrlis@sbrella.com>
+
+subst.c
+ - string_list_pos_params: if we are expanding $* in a context where
+ IFS is null and expand_no_split_dollar_star is set (no word
+ splitting), use string_list_dollar_star to force the positional
+ parameters to be joined without first separating them into a list
+ and attempting to join them later. Fixes problem with constructs
+ like ${var=${*:1}}, pointed out by Grisha Levit
+ <grishalevit@gmail.com>. This undoes change to @Q from 4/7, so
+ list_transform change reverted
+
+ 4/19
+ ----
+builtins.h
+ - added BUILTINS_H include guard
+
+examples/loadables/Makefile.in
+ - install loadables.h as sample include file, along with Makefile.inc
+
+ 4/23
+ ----
+bashline.c
+ - maybe_make_readline_line: don't try to do anything if the passed
+ NEW_LINE is NULL. Report from Jaren Stangret <sirjaren@gmail.com>,
+ fix from Eduardo Bustamante <dualbus@gmail.com>
+
+ 4/24
+ ----
+parse.y
+ - xparse_dolparen: if the current token (the last thing read_token
+ returned to yylex) is shell_eof_token, assume that it is the new
+ bison lookahead token and clear it. Fixes bug reported by
+ Werner Fink <werner@suse.de>
+
+ 4/25
+ ----
+doc/{bash.1,bashref.texi}
+ - cmdhist: clarify that this option only has an effect if history is
+ enabled. Suggested by Matthew Braun <matthew@powerplug.com>
+
+ 4/26
+ ----
+jobs.c
+ - wait_for: if a non-interactive shell with job control enabled (set -m)
+ detects that a foreground job died due to SIGINT, act as if the shell
+ also received the SIGINT. Prompted by an austin-group-l discussion
+ - waitchld: run SIGCHLD trap for each child exited even if job control
+ is not enabled when in Posix mode. Prompted by an austin-group-l
+ discussion
+
+ 4/27
+ ----
+lib/readline/histfile.c
+ - read_history_range: if the history file is empty, free the history
+ filename before returning. Report and fix from Eduardo Bustamante
+ <dualbus@gmail.com>
+
+lib/readline/bind.c
+ - rl_parse_and_bind: make sure there is something, even if it's a
+ quoted empty string, before the `:' in a key binding. Report from
+ Eduardo Bustamante <dualbus@gmail.com>
+ - rl_parse_and_bind: if the right side of a key binding starts with a
+ quote, make sure there's a matching close quote before treating it
+ as a macro definition
+ - rl_translate_keyseq: if a key sequence ends with \C- or \M- (or
+ \C-\M-) make sure we break out of the loop if moving to the character
+ to be translated is a NUL. Old code did this only in the \C-\M-
+ case. Report from Eduardo Bustamante <dualbus@gmail.com>
+
+ 4/28
+ ----
+lib/glob/sm_loop.c
+ - GMATCH: implement a clever technique from glibc that avoids
+ backtracking past a `*' if we've already chosen to use it and need
+ matches beyond it. Look at https://research.swtch.com/glob for a
+ longer explanation. This results in a significant speedup for globs
+ with multiple instances of `*', especially with more than 4.
+
+ 5/2
+ ---
+lib/readline/bind.c
+ - rl_translate_keyseq: make sure a trailing backslash in the key
+ sequence is preserved. Report from Eduardo Bustamante
+ <dualbus@gmail.com>
+
+ 5/3
+ ---
+builtins/builtin.def
+ - builtin_builtin: make sure to set this_shell_builtin to the builtin
+ command being executed, overwriting `builtin'. Leave
+ last_shell_builtin alone. Fixes bug reported by Luiz Angelo Daros
+ de Luca <luizluca@gmail.com>
+
+jobs.c
+ - waitchld: modify change of 4/26 to run SIGCHLD traps even if job
+ control is not enabled
+
+lib/readline/misc.c
+ - rl_get_previous_history: if we didn't find a previous history entry
+ to use, call rl_maybe_unsave_line before returning because we
+ aren't going to use the saved line, and it has the same undo list
+ as rl_undo_list. Fixes fuzzing bug reported by Eduardo Bustamante
+ <dualbus@gmail.com>
+ - rl_get_{next,previous}_history: if there is no history list, don't
+ bother doing anything
+
+ 5/4
+ ---
+expr.c
+ - popexp: if we error out due to stack underflow, make sure to zero out
+ expression and lasttp; they may contain undefined values we don't
+ want evalerror to print
+ - expr_unwind: handle expr_depth decrementing below zero
+ - evalexp: restore the old value of evalbuf even if we error and
+ longjmp; we may have called evalexp recursively
+ - evalerror: make sure expression is non-NULL before trying to print it.
+ These fix fuzzing bug reported by Eduardo Bustamante
+ <dualbus@gmail.com>, happens only in cases where we don't longjmp on
+ a fatal expansion error
+
+ 5/5
+ ---
+parse.y
+ - read_token_word: when checking for a word that's a target of a
+ redirection, we can potentially call valid_array_reference, which
+ can end up calling the parser recursively. If it does that, we
+ need to make sure that yylval.word doesn't change, so we set it back
+ to the_word. Fixes another fuzzing bug
+
+parse.y
+ - arith_for_command: make syntax errors in the arithmetic for command
+ (signaled by make_arith_for_command returning NULL) more like
+ language syntax errors by calling YYERROR. Avoids complicated
+ attempts at error recovery and dealing with NULL returns from
+ command productions. Pointed out by Eduardo Bustamante
+ <dualbus@gmail.com>
+
+lib/readline/kill.c
+ - _rl_copy_to_kill_ring: make sure the current slot in the kill ring
+ has something in it, even if the last command was a kill, before
+ trying to modify it. Another fuzzing bug
+
+ 5/6
+ ---
+builtins/read.def
+ - read_builtin: if we see a backslash when not in raw mode (not -r),
+ only back up `i' when we read the next character if we know we added
+ a CTLESC the last time through the loop (skip_ctlesc == 0),
+ especially if i == 0. Another fuzzing bug from Eduardo Bustamante
+ <dualbus@gmail.com>
+
+ 5/8
+ ---
+builtins/read.def
+ - read_mbchar: handle zreadn/zreadc/zread returning EOF in the middle
+ of an incomplete multibyte sequence. Fixes another fuzzing bug
+ - read_builtin: use mb_cur_max instead of constant 4 when deciding
+ whether the next character can exceed the number of bytes available
+ in input_string
+
+lib/readline/input.c
+ - MinGW: include <conio.h> before <io.h>. Fix from Eli Zaretskii
+ <eliz@gnu.org>
+
+builtins/read.def
+ - read_builtin: if we get input from readline, we need to get the
+ remainder of a multibyte character from rlbuf instead of calling
+ read_mbchar. Bug reported by Eduardo Bustamante <dualbus@gmail.com>
+
+ 5/9
+ ---
+parse.y
+ - token_is_assignment: use the allocated buffer approach in all cases,
+ not just if we're not using bash malloc. This avoids the assignment
+ to t[i+1] writing beyond the end of the allocated token if
+ i == token_buffer_size - 1. Another fuzzing bug
+ - xparse_dolparen: if parse_string returns < 0, we clear out the
+ current shell_input_line before performing a longjmp, since we're
+ abandoning parsing of this command. This is consistent with how
+ the parser resynchronizes after other syntax errors
+ - GRAMMAR: add 'error yacc_EOF' production to handle a syntax error
+ that's immediately followed by an EOF after resynchronization.
+ Fixes another fuzzing bug
+
+ 5/10
+ ----
+lib/readline/text.c
+ - _rl_set_mark_at_pos: don't let the mark be set to a position < 0.
+ Fixes a fuzzing bug
+ - rl_exchange_point_and_mark: don't do anything if the mark is already
+ less than 0
+
+ 5/12
+ ----
+lib/readline/rlmbutil.h
+ - wchar_t: make sure this is defined as int if multibyte characters
+ aren't supported
+
+lib/readline/text.c
+ - _rl_change_case: don't force use of ascii toupper and tolower if
+ isascii returns true; it's not defined to work on characters outside
+ the 0..255 (really 0..127) range. Bug reported by
+ Eduardo Bustamante <dualbus@gmail.com>
+
+ 5/14
+ ----
+lib/readline/text.c
+ - rl_change_case: handle case where the old and new characters (after
+ the case change) are not the same number of bytes. Pointed out in
+ http://lists.gnu.org/archive/html/bug-bash/2015-01/msg00091.html
+
+lib/readline/display.c
+ - expand_prompt: use `ret' as first parameter to _rl_find_prev_mbchar
+ since that needs the base of the array. Bug from fuzzing reported by
+ Eduardo Bustamante <dualbus@gmail.com>
+
+ 5/15
+ ----
+execute_cmd.c
+ - execute_simple_command, execute_arith_command, execute_cond_command,
+ execute_arith_for_command: adjust the line number when executing a
+ function interactively so that the first command in the function is
+ line 1, not line 0, as Posix requires. Reported by Robert Elz
+ <kre@bmunnari.OZ.AU>
+ - execute_simple_command, execute_arith_command, execute_cond_command,
+ execute_arith_for_command: ony adjust the line number when executing
+ a function interactively if we are not sourcing a file
+ (sourcelevel == 0), so this is consistent everywhere we adjust the
+ line number
+
+ 5/16
+ ----
+bashline.c
+ - bash_directory_completion_hook: when calling split_at_delims to see
+ whether a ${ or $( is closed, use the right value for `start' relative
+ to the substring beginning at `$'. Bug from fuzzing reported by
+ Eduardo Bustamante <dualbus@gmail.com>
+
+expr.c
+ - expassign: if the call to expcond generates a syntax error in a
+ context when the shell won't longjmp (like when evaluating $PS1),
+ it will end up NULL and we need to catch it before calling strlen.
+ Bug from fuzzing reported by Eduardo Bustamante <dualbus@gmail.com>
+
+examples/bash-completion/
+ - new place to include a current or recent version of the
+ bash-completion package
+
+ 5/19
+ ----
+lib/readline/display.c
+ - CHECK_INV_LBREAKS: new versions for multibyte and single-byte chars,
+ so the multibyte version can check and increase the size of
+ line_state_invisible->wbsize and line_state_invisible->wrapped_line,
+ callers who call CHECK_INV_LBREAKS must update
+ wrapped_line[newlines] when in HANDLE_MULTIBYTE mode. Fuzzing bug
+ reported by Eduardo Bustamante <dualbus@gmail.com>
+
+ 5/24
+ ----
+lib/readline/search.c
+ - _rl_nsearch_callback,noninc_search: handle _rl_search_getchar
+ returning -1 (EOF or read error) by aborting the search.
+ Fuzzing bug reported by Eduardo Bustamante <dualbus@gmail.com>
+
+ 5/25
+ ----
+variables.c
+ - localvar_inherit: new variable, controlled by shopt localvar_inherit
+ option
+ - make_local_variable: if localvar_inherit is set, the new local
+ variable inherits a previous scope's variable's value, attributes
+ (except nameref), and dynamic variable information. If a local
+ variable inherits a value, the local is not invisible
+
+builtins/shopt.def
+ - localvar_inherit: new option
+
+doc/{bash.1,bashref.texi}
+ - localvar_inherit: document new shopt option
+
+ 5/29
+ ----
+lib/readline/readline.c
+ - _rl_subseq_result: only return -1 and back up the chain if we are
+ dealing with a result (r) that's already < 0 and we are at the end
+ of a multi-key sequence. Otherwise, a failing readline command (e.g.,
+ delete-char at the end of a line) could cause this code to be
+ executed. Report from Nuzhna Pomoshch <nuzhna_pomoshch@yahoo.com>
+
+lib/readline/histfile.c
+ - read_history_range: if the file isn't a regular file, return an
+ error. Bug report from Eduardo Bustamante <dualbus@gmail.com>,
+ relaying from IRC
+
+ 5/30
+ ----
+variables.c
+ - set_pwd: if in Posix mode, and PWD appears in initial environment as
+ an absolute pathname to the current directory, set PWD to the result
+ of canonicalizing the environment value, or to the physical path if
+ canonicalization fails. From a suggestion by Eduardo Bustamante
+ <dualbus@gmail.com>
+
+ 5/31
+ ----
+builtins/read.def
+ - read_builtin: if -n or -N option is supplied with a 0 argument,
+ don't attempt to read any characters; bail out right away. Reported
+ by Eduardo Bustamante <dualbus@gmail.com>, relaying from IRC
+
+ 6/3
+ ---
+config.h.in
+ - HAVE_STRUCT_STAT_ST_ATIM_TV_NSEC: add define, used by stat-time.h
+ to construct a timespec from struct stat. Report and fix from
+ Siteshwar Vashisht <svashisht@redhat.com>
+
+variables.h
+ - att_regenerate: new internal variable attribute: if set, regenerate
+ dynamic variable's value when it's exported and we are creating the
+ export environment
+
+variables.c
+ - init_dynamic_variables: LINENO, EPOCHSECONDS, EPOCHREALTIME: set
+ internal regenerate attribute for these dynamic variables
+ - make_env_array_from_var_list: if a dynamic variable has the
+ regenerate attribute set, call the dynamic value function to generate
+ an updated value before placing it in the environment. From a report
+ about exporting LINENO from Robert Elz <kre@bmunnari.OZ.AU>
+
+ 6/4
+ ---
+lib/glob/sm_loop.c
+ - BRACKMATCH: at the matched: label, make sure we get the bracket
+ character we're looking for (char class, collating symbol. etc.)
+ before we decrement the count of braces we're looking for. Eventually
+ we could do something about badly-formed bracket expressions
+
+ 6/7
+ ---
+lib/readline/histlib.h
+ - strchr: only declare if __STDC__ is not defined, since we already
+ include <string.h>. Report from Chi-Hsuan Yen <yan12125@gmail.com>
+
+ 6/9
+ ---
+lib/readline/display.c
+ - update_line: when wrapping lines with multibyte chars at the end of
+ the new line being wrapped, make sure we copy the NULL byte in old
+ when moving the contents of old around. Fuzzing bug reported by
+ Eduardo Bustamante <dualbus@gmail.com>
+
+lib/readline/mbutil.c
+ - _rl_get_char_len: use MB_CUR_MAX in the call to mbrlen; there's no
+ need to look at the rest of the string when we're just interested in
+ the length of a single character
+
+ 6/12
+ ----
+jobs.c
+ - wait_for: when waiting for the next child to exit (ANY_PID), make
+ sure to restore the old SIGINT handler before returning. Fixes
+ https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=864649
+
+ 6/14
+ ----
+lib/readline/display.c
+ - expand_prompt: make sure `ret' is NULL-terminated before calling
+ _rl_prev_mbchar_internal, since that calls strlen, which requires
+ the passed string to be NULL-terminated. Another fuzzing bug.
+
+ 6/15
+ ----
+lib/readline/isearch.c
+ - _rl_isearch_fini: use rl_replace_line instead of strcpy so rl_end
+ gets set right
+ - _rl_isearch_fini: after restoring rl_point, call _rl_fix_point so
+ we don't set rl_point > rl_end. Fixes a fuzzing bug
+
+lib/readline/mbutil.c
+ - _rl_find_next_mbchar_internal: if _rl_adjust_point returns < 0,
+ just punt, treat the value as a byte, and advance point by 1
+
+lib/readline/util.c
+ - rl_tilde_expand: rearrange code in the whitespace loop so that
+ `start' gets tested first and we don't try to dereference
+ rl_line_buffer[-1]. Another fuzzing bug from dualbus@gmail.com
+
+bashline.c
+ - bash_dequote_filename: make sure that pointers that are used as
+ indices into sh_syntaxtab are cast to unsigned char first, to
+ avoid problems with signed chars > 128. Fixes a fuzzing bug.
+
+lib/readline/kill.c
+ - _rl_copy_to_kill_ring: when reallocating the kill ring, make sure to
+ allocate one more than the max number of kills so the loop that
+ copies the kill ring entries down runs right (and to mirror the
+ initial allocation). Fixes a fuzzing bug.
+
+lib/readline/isearch.c
+ - _rl_isearch_dispatch: make sure that cxt->sline_index never goes < 0
+ even when searching a line in reverse. Fixes a fuzzing bug
+
+ 6/16
+ ----
+parse.y
+ - xparse_dolparen: short-circuit immediately if passed a empty string
+
+lib/readline/display.c
+ - update_line: when wrapping multibyte characters, make sure we deal
+ with WCWIDTH returning -1. Fixes a fuzzing bug
+
+ 6/17
+ ----
+execute_cmd.c
+ - execute_coproc: make sure `invert' is set before trying to use it
+ when returning failure on invalid coproc name. Report and fix from
+ Eduardo Bustamante <dualbus@gmail.com>
+ - execute_command_internal: make sure execute_coproc sets
+ last_command_exit_value if it returns failure, so an invalid name
+ can set $? = 1. Report and fix from Eduardo Bustamante
+ <dualbus@gmail.com>
+
+lib/readline/display.c
+ - update_line: make sure all references to `wrapped_line' are wrapped
+ with #ifdef HANDLE_MULTIBYTE. Report and fix from Eduardo Bustamante
+ <dualbus@gmail.com>
+
+lib/readline/vi_mode.c
+ - _rl_vi_change_char: don't use rl_point++ when you mean to move
+ forward a character; use _rl_vi_append_forward to account for
+ multibyte characters and take vi end of line handling into account
+ - _rl_vi_last_replacement: now an array of chars whether we are
+ using multibyte chars or not. If we're not, the character we read
+ to use as the replacement is saved as the first element of the array
+ - rl_vi_change_char,_rl_vi_callback_change_char: changes to deal with
+ _rl_vi_last_replacement being an array. Fixes bug reported by
+ Eduardo Bustamante <dualbus@gmail.com>
+
+lib/readline/mbutil.c
+ - _rl_get_char_len: look at at most MB_CUR_MAX characters, but maybe
+ fewer if the length of the string is less
+
+builtins/bind.def
+ - unbind_keyseq: new function for the -r option; checks whether the
+ key sequence is actually bound before trying to bind it to NULL.
+ Partial fix for https://savannah.gnu.org/support/?109329
+
+parse.y
+ - augment `error yacc_EOF' production to call YYABORT in non-interactive
+ shells or calls to parse_and_execute (eval, command substitution,
+ etc.) Fixes bug reported by Martijn Dekker <martijn@inlv.org>
+
+ 6/19
+ ----
+bashline.c
+ - edit_and_execute_command: don't add rl_line_buffer to the history
+ list if it's empty; consistent with how other code treats an empty
+ line
+
+execute_cmd.c
+ - execute_builtin: make sure to preserve the temporary env across the
+ execution of the `read' builtin or `fc' builtin if HISTORY is
+ defined, in case `read -e' calls edit-and-execute-command. Should
+ have no side effects. Reported by Eduardo Bustamante
+ <dualbus@gmail.com>
+
+general.c
+ - line_isblank: new function: returns true if passed string is composed
+ entirely of blanks
+
+general.h
+ - line_isblank: new extern declaration
+
+parse.y
+ - history_delimiting_chars: return "" for a blank line, since there's
+ nothing to delimit with `;'
+
+ 6/21
+ ----
+jobs.c
+ - wait_for: make sure to call restore_sigint_handler before returning
+ if we return out of the loop due to no children. Report from
+ Eduardo Bustamante <dualbus@gmail.com>
+
+subst.c
+ - expand_word_internal: if split_on_spaces is set, and the word is
+ unquoted, and IFS is null, split the results of the previous steps
+ on $' \t\n' instead of just ' '. This relies on the previous steps
+ quoting the portions of the word that should not be split. Fixes
+ bug reported by Kevin Brodsky <corax26@gmail.com>
+
+expr.c
+ - evalexp: after running expr_unwind, make sure we reset expr_depth
+ to 0 for the next call
+ - expr_streval: if after a call to get_array_value (which can call
+ the expression evaluator recursively) we discover that expr_depth
+ is less than it was before we called it, we assume there has been
+ some kind of error and an expr_unwind, so we treat it as an
+ error and either longjmp back to the expression top level or return
+ 0 immediately. Fixes bug reported by Eduardo Bustamante
+ <dualbus@gmail.com>
+
+ 6/23
+ ----
+doc/{bash.1,bashref.texi}
+ - add text noting that $* and ${array[*]} (unquoted) can also expand
+ to multiple words
+
+ 6/29
+ ----
+general.[ch]
+ - default_columns: new function, returns the value of COLUMNS, or
+ refreshes it if check_window_size is set and COLUMNS is unset. By
+ default, it returns 80
+
+execute_cmd.c
+ - select_query: use default_columns() instead of fetching value of
+ COLUMNS directly
+
+builtins/help.def
+ - show_builtin_command_help: use default_columns() instead of fetching
+ value of COLUMNS directly
+
+ 6/30
+ ----
+builtins/read.def
+ - read_builtin: call QUIT during the read loop, just in case we get a
+ signal we should act on that didn't cause read to be interrupted.
+ Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1466737
+ - read_builtin: if -n or -N is supplied with a 0 argument, try a zero-
+ length read to detect errors and return failure if that read returns
+ a value < 0. Suggested by dualbus@gmail.com
+
+ 7/4
+ ---
+bashhist.c
+ - maybe_add_history: keep track of whether the current line is a shell
+ comment, even if we're not adding it because it's in a multi-line
+ command, so we use appropriate delimiters between it and any
+ subsequent lines. Fixes bug reported by Grisha Levit
+ <grishalevit@gmail.com> back on 2/28/2017
+
+bashline.c
+ - operate_and_get_next: if given an explicit argument, use that to
+ choose which line in the history to use.
+
+lib/readline/doc/rluser.texi,doc/bash.1
+ - operate-and-get-next: document new effect of explicit numeric arg
+
+lib/readline/complete.c
+ - fnprint: make sure print_len is initialized before using it on
+ systems without multibyte character support. Report and fix from
+ Juan Manuel Guerrero <juan.guerrero@gmx.de>
+
+ 7/6
+ ---
+builtins/printf.def
+ - PRETURN,printf_builtin: check variable returned by bind_printf_variable,
+ return failure if that indicates we can't perform an assignment
+ because the variable is marked readonly or noassign. Fixes bug
+ reported by Arnaud Gaillard <arnaud.mgaillard@gmail.com>
+
+ 7/7
+ ---
+lib/readline/text.c
+ - rl_quoted_insert: new feature: a negative argument means to insert
+ the next -COUNT characters using quoted-insert. Original feature
+ from Jason Hood <jadoxa@yahoo.com.au>. Still needs work on
+ redisplay
+ - _rl_insert_next_callback: implement support for negative arguments
+ similar to rl_quoted_insert: we just insert one at a time and keep
+ increasing the count until it hits 0
+
+lib/readline/misc.c
+ - _rl_arg_callback: if the return value from _rl_arg_dispatch indicates
+ we should keep reading a numeric argument, update the message with
+ the new arg value
+
+ 7/8
+ ---
+lib/readline/signals.c
+ - _rl_handle_signal: make sure all uses of any of the job control
+ signals are protected by a check for SIGTSTP being defined. Report
+ from Juan Manuel Guerrero <juan.guerrero@gmx.de>
+
+ 7/11
+ ----
+lib/readline/vi_mode.c
+ - rl_vi_replace: when making the new keymap for vi replacement mode,
+ make sure that ANYOTHERKEY is set correctly, otherwise some input
+ will cause _rl_dispatch to return -2 to the top level. Fixes fuzzing
+ bug reported by Ben Wong <benjamin.a.wong@gmail.com>
+
+ 7/19
+ ----
+builtins/read.def
+ - struct ttsave: make the attrs member a struct, not a pointer, to force
+ a structure copy that will survive a longjmp to another context.
+ Leaving it as a pointer to a local struct is not portable
+
+ 9/10
+ ----
+execute_cmd.c
+ - execute_builtin: make sure that we set up the unwind-protect for
+ pop_scope (temporary_env is non-zero) so that the temporary env
+ is propagated to the current environment only for special builtins
+ (source/eval/unset) not run by the command builtin
+ (flags & CMD_COMMAND_BUILTIN == 0). Fixes bug reported by
+ Martijn Dekker <martijn@inlv.org>
+
+ 9/17
+ ----
+builtins/printf.def
+ - asciicode: don't use mblen to check whether or not a character is a
+ valid multibyte character; use mbtowc right away and then inspect
+ the return value. Fixes bug reported by Stephane Chazelas
+ <stephane.chazelas@gmail.com>
+
+ 9/27
+ ----
+{jobs,subst}.h
+ - move declaration of last_command_subst_pid to subst.h, since it's
+ declared in subst.c and doesn't depend on job control. Reported by
+ Martijn Dekker <martijn@inlv.org>
+
+builtins/read.def,variables.c
+ - fixed a couple of problems (READLINE, ARRAY_VARS) that prevented
+ the minimal config from building
+
+ 10/1
+ ----
+parse.y
+ - special_case_tokens: if we are returning DO in a case where the last
+ two tokens read are `FOR' and `WORD' make sure we decrement
+ expecting_in_token. Fixes bug reported by Martijn Dekker
+ <martijn@inlv.org>
+
+ 10/4
+ ----
+subst.c
+ - expand_string_for_rhs: now takes an additional PFLAGS argument from
+ its caller (one so far); passes that through to call_expand_word_internal
+ as W_ASSIGNRHS. Fixes bug reported by Martijn Dekker
+ <martijn@inlv.org>
+ - expand_string_for_rhs: expanding b in ${a[:]=b} is now done as if
+ b were the rhs of an assignment statement. This means that splitting
+ isn't done, but tilde expansion is performed as if the statement
+ were `a=b'. Fixes splitting bug with $* reported by Martijn Dekker
+ <martijn@inlv.org>
+
+command.h
+ - W_NOASSNTILDE: new word flag, means to not perform tilde expansion
+ following a `:' even if the word has the W_ASSIGNRHS flag enabled
+
+subst.c
+ - expand_word_internal: if we see a `:' with the W_NOASSNTILDE flag
+ set, just add the character and inhibit any subsequent tilde
+ expansion. Currently not set anywhere, but it could be set in
+ expand_string_for_rhs to satisfy its peculiar semantics
+
+ 10/6
+ ----
+lib/sh/casemod.c
+ - sh_modcase: convert even single-byte wide characters to wide upper
+ or lowercase equivalents to accommodate locales where single-byte
+ characters have multibyte upper and lower case conversions. Bug
+ reported by Stephane Chazelas <stephane.chazelas@gmail.com>
+
+ 10/7
+ ----
+doc/{bash.1,bashref.texi}
+ - slight changes to the description of command_not_found_handle to
+ clarify that it's executed in a separate execution environment,
+ just like if the command was found. Reported by Martijn Dekker
+ <martijn@inlv.org>
+
+ 10/8
+ ----
+lib/readline/doc/{history.3,hstech.texi}
+ - history_get: clarify the range of valid values for the OFFSET
+ argument. From a report by Kevin Ryde <user42_kevin@yahoo.com.au>
+
+ 10/21
+ -----
+subst.c
+ - expand_string_for_rhs: set W_NOASSNTILDE if the operator is `='
+ for backwards compatibility with bash-4.4
+
+shell.c
+ - main: set positional parameters before running the startup files,
+ so the startup files can inspect $@. Often-requested feature,
+ most recently from Stephane Chazelas <stephane.chazelas@gmail.com>
+
+ 10/27
+ -----
+doc/{bash.1,bashref.texi}
+ - Arrays: add some clarifying language to make it clear that array
+ references that don't use the ${a[s]} syntax are subject to
+ globbing when passed as arguments to commands such as unset, and
+ should be quoted for safety. Change prompted by a report from
+ Eli Barzilay <eli@barzilay.org>
+
+parse.y
+ - parse_comsub: make sure we don't run off the end of the `ret'
+ buffer when checking for the here doc delimiter. Report from
+ Jakub Wilk <jwilk@jwilk.net>, the result of a fuzzing test. Pointer
+ to place for the fix from Eduardo Bustamante <dualbus@gmail.com>
+
+ 10/30
+ -----
+builtins/pushd.def
+ - get_directory_stack: make sure the current directory (element 0 of
+ the stack) is passed to polite_directory_format under the same
+ conditions as the rest of the stack entries (flags & 1). Otherwise
+ something like `cd ${DIRSTACK[0]}' will fail. Fixes bug reported
+ by Steve Jones <sjml@slohj.org>
+
+builtins/declare.def
+ - declare_internal: when checking for a `[' to see whether or not this
+ is an array variable declaration (declare -a foo[12]), make sure
+ we don't do the check if we're just dealing with shell functions.
+ Bug and pointer to fix from PJ Eby <pje@telecommunity.com>
+
+ 11/1
+ ----
+parse.y
+ - parse_comsub: if we read a four-character word followed by a break
+ character, and that word is not one of the reserved words, set
+ lex_rwlen to 0 since we are no longer in a reserved word. It only
+ hurts if another break character immediately follows, so that test
+ succeeds again. Turn off the RESWDOK flag only if it's not a shell
+ metacharacter, too. Fixes bug reported by Kjetil Torgrim Homme
+ <kjetilho@scribus.ms.redpill-linpro.com>
+
+ 11/3
+ ----
+lib/readline/isearch.c
+ - _rl_isearch_dispatch: if we are searching in reverse order, let
+ sline_index go to -1 to avoid searching the same line twice. It
+ gets reset right after that, so there's no danger of indexing into
+ the history line with a negative index.
+
+ 11/7
+ ----
+execute_cmd.c
+ - time_command: only restore command->flags if CODE indicates we didn't
+ perform a longjmp back to top_level. If we did, `command' has already
+ been freed. Fixes bug reported on savannah by
+ ukuvbu oibws <xyzdr4gon333@googlemail.com>
+ https://savannah.gnu.org/support/?109403
+
+ 11/10
+ -----
+lib/sh/unicode.c
+ - u32cconv: make sure to initialize localconv to -1 (error) in case
+ we switch from a utf-8 locale to something else and call
+ iconv_close. Report from Egmont Koblinger <egmont@gmail.com>; fix
+ from Eduardo Bustamante <dualbus@gmail.com>
+
+ 11/16
+ -----
+subst.c
+ - parse_comsub: istring_index should be a size_t to avoid integer
+ overflow when allocating large pieces of memory. Report and fix
+ from Siteshwar Vashisht <svashisht@redhat.com>, originally based on
+ http://lists.gnu.org/archive/html/bug-bash/2017-11/msg00047.html
+
+ 11/24
+ -----
+lib/readline/bind.c
+ - rl_empty_keymap: new public function, returns non-zero if there are
+ no keys bound in the keymap passed as an argument
+
+lib/readline/readline.h
+ - rl_empty_keymap: new public extern declaration
+
+lib/readline/doc/rltech.texi
+ - rl_empty_keymap: document new function
+
+lib/readline/bind.c
+ - rl_generic_bind: keep track of the previous keymap and previous
+ index in a multi-key key sequence so we can remove an empty terminal
+ keymap and remove a previous override to ANYOTHERKEY. Right now,
+ this only works for one previous keymap level. Inspired by a report
+ from Clark Wang <dearvoid@gmail.com>
+
+ 11/25
+ -----
+lib/readline/display.c
+ - _rl_update_final: make sure that computing the length of the visible
+ bottom line (botline_length) takes any invisible characters in the
+ prompt into account (woff). This value is used as part of the check
+ whether or not we want to remove the line-wrapping indicator on
+ terminals like xterm
+ - _rl_update_final: when removing the line-wrapping indicator on xterm,
+ make sure we take invisible characters in the prompt (woff) into
+ account when computing the character in the rightmost physical
+ screen position. Fixes cosmetic line-wrapping issue reported by
+ Egmont Koblinger <egmont@gmail.com>
+
+ 11/27
+ -----
+lib/sh/ufuncs.c
+ - fsleep: if using select to implement sleep, use restart after
+ a signal interrupts the select and handle pending signals. If
+ pselect is available, use that instead and block SIGCHLD while
+ pselect is executing.
+ Fixes report from Thiruvadi Rajaraman <trajaraman@mvista.com>
+
+ 11/28
+ -----
+lib/readline/signals.c
+ - rl_check_signals: new public function: a wrapper for RL_CHECK_SIGNALS
+ that can be used by applications that install a wrapper for
+ rl_getc (rl_getc_function) so they can respond to signals that
+ arrive while waiting for input in the same way as rl_getc. The app
+ can use rl_pending_signal() to get the most recently-received
+ signal. From a discussion about Gnuplot initiated by
+ Rin Okuyama <rokuyama@rk.phys.keio.ac.jp>
+
+lib/readline/readline.h
+ - rl_check_signals: new extern declaration
+
+lib/readline/doc/rltech.texi
+ - rl_check_signals: document new public function
+
+variables.c
+ - set_pwd: test for and force an inherited OLDPWD to be a directory
+ if OLDPWD_CHECK_DIRECTORY is defined in config-top.h (it is by
+ default). Issue raised by Mikulas Patocka <mikulas@twibright.com>
+
+config-top.h
+ - OLDPWD_CHECK_DIRECTORY: new define, defined to 1 by default
+
+ 11/29
+ -----
+Makefile.in
+ - pathexp.o: add dependencies on libintl.h. Reported by
+ Ross Burton <ross@burtonini.com>
+
+ 12/1
+ ----
+lib/sh/ufuncs.c
+ - fsleep: add blocking and releasing SIGCHLD using sigprocmask
+ around call to select(2) even if pselect(2) is not available
+
+ 12/3
+ ----
+execute_cmd.c
+ - coproc_setstatus: new utility function, take a pointer to a coproc
+ and a status and mark the coproc as dead and having been reaped
+ with that status. Used by child processes who want to invalidate
+ the coproc's pid
+
+ 12/5
+ ----
+subst.c
+ - process_substitute: instead of just having a flag to denote that a
+ file descriptor is connected to a process substitution, store the
+ pid of the child process (still needs work)
+ - find_procsub_child,set_procsub_status,reap_procsubs: utility
+ functions to allow the job control code to detect that a process
+ substitution child has died and deallocate the file descriptors
+ or pathname
+
+subst.h
+ - find_procsub_child,set_procsub_status,reap_procsubs: extern
+ declarations
+
+nojobs.c
+ - set_pid_status: if we are reaping a process substitution, call
+ set_procsub_status to record the fact
+
+jobs.c
+ - waitchld: if we are reaping a process substitution, call
+ set_procsub_status to record the fact. Doesn't really do anything
+ yet
+
+ 12/6
+ ----
+execute_cmd.c
+ - execute_command: don't call unlink_fifo_list if we're in the
+ middle of executing a list of commands (executing_list != 0).
+ Partial fix for bug reported by Stephane Chazelas
+ <stephane.chazelas@gmail.com>
+ - execute_simple_command: don't close process substitution file
+ descriptors in the parent after forking children in a pipeline;
+ defer until pipeline completes, since parent may have inherited
+ fds from a calling scope (e.g., via `.'). EXPERIMENTAL
+ - execute_disk_command: don't close process substitution file
+ descriptors in the parent after forking the child. EXPERIMENTAL
+
+ 12/7
+ ----
+doc/bash.1,lib/readline/doc/{readline.3,rluser.texi}
+ - show-mode-in-prompt: clarify that this must be enabled before the
+ vi and emacs mode strings are prefixed to the prompt. Report from
+ Rob Foehl <rwf@loonybin.net>
+
+ 12/8
+ ----
+execute_cmd.c
+ - execute_coproc: don't warn about an existing coproc if the write and
+ read file descriptors are unusable (-1), as they will be after a
+ call to coproc_closeall in a subshell. Fixes spurious warning
+ reported by Tobias Hoffmann <lfile-list@thax.hardliners.org>
+
+jobs.c
+ - bgp_resize: avoid overflow when calculating the new size if the
+ child process limit is something ridiculously large. Report and
+ fix from Natanael Copa <ncopa@alpinelinux.org>
+
+execute_cmd.c
+ - execute_builtin_or_function: if we return via the return builtin, we
+ don't get a chance to free the saved fifo list, so add an unwind-
+ protect to make sure it happens. Part of fix for leak reported by
+ Oyvind Hvidsten <oyvind.hvidsten@dhampir.no>
+ - execute_command_internal: if we unwind via the return builtin (e.g.,
+ we execute a function which then returns via the return builtin to
+ a previous function), we don't free the saved fifo list, so add an
+ unwind-protect to make sure the free happens. Rest of fix for leak
+ reported by Oyvind Hvidsten <oyvind.hvidsten@dhampir.no>
+
+subst.c
+ - add_fifo_list: for the FIFO case (no /dev/fd), make sure we
+ initialize the new members of the fifo list after xrealloc
+
+ 12/9
+ ----
+subst.c
+ - dev_fd_list: now a list of type `pid_t' so we can store process
+ associated with the pipe fd into the right element; change all
+ necessary functions (copy_fifo_list, add_fifo_list)
+ - process_substitute: store pid of child process into the right
+ index in dev_fd_list after opening the pipe. Process ID of -1 means
+ process has been reaped and fd needs to be closed (or FIFO needs to
+ be unlinked); value of 0 means slot is unused
+
+jobs.c
+ - wait_for_background_pids: call reap_procsubs to make sure we clean
+ up all reaped process substitutions before trying to wait for
+ everything (still needs work)
+
+tests/{run-procsub,procsub.{tests,right}}
+ - new file, tests of process substitution that have failed in the past
+
+ 12/10
+ -----
+jobs.c
+ - wait_for_background_pids: warn if one of the jobs is stopped
+
+subst.c
+ - wait_procsubs: new function, call wait_for for every "live" process
+ substitution
+
+jobs.c
+ - wait_for_background_pids: call wait_procsubs to reap any living
+ process subsitutions
+
+ 12/13
+ -----
+lib/readline/bind.c
+ - parser_if: add support for testing the readline version, using the
+ full set of arithmetic comparison operators (and supporting both
+ = and ==), using version numbers of the form major[.[minor]]
+
+ 12/14
+ -----
+subst.[ch]
+ - string_list_dollar_star: now takes QUOTED and PFLAGS arguments like
+ string_list_dollar_at, changed all callers. Not used yet.
+
+ 12/16
+ -----
+subst.c
+ - param_expand: broke out cases of expanding unquoted (quoted == 0)
+ $* on the rhs of an assignment statement (pflags & PF_ASSIGNRHS)
+ with various values of IFS (unset, null, set to non-null value) to
+ capture the expansion subtleties. From a report back on 11/24 by
+ Martijn Dekker <martijn@inlv.org>
+
+ 12/17
+ -----
+array.h
+ - set_element_value: new define, sets array element AE to VALUE
+
+variables.c
+ - set_pipestatus_array: use set_element_value where appropriate
+
+ 12/18
+ -----
+subst.c
+ - parameter_brace_find_indir: when expanding the indirect parameter
+ to find the eventual variable name, we don't perform word splitting.
+ Make sure this does the right thing for * and @. Fixes bug
+ reported by isabella parakiss <izaberina@gmail.com>
+
+ 12/19
+ -----
+doc/{bash.1,bashref.texi}
+ - indirect expansion: make sure to note that the value of the indirect
+ variable does not undergo word splitting as one of its expansions,
+ as in fix from 12/18
+
+ 12/22
+ -----
+subst.c
+ - parameter_brace_expand_rhs: make sure the value this function returns
+ when OP is `=' is quoted appropriately, as the callers expect. More
+ changes from Posix interp 221. Fixes report from Martijn Dekker
+ <martijn@inlv.org>
+
+variables.c
+ - assign_hashcmd: if running in a restricted shell, make sure the
+ target of the hash assignment can be found via a $PATH search, to
+ prevent users assigning commands to the hash table they would not
+ ordinarily have access to. Fixes issue raised by Drew Parker
+ <andrew.s.parker2@gmail.com>
+
+builtins/hash.def
+ - hash_builtin: if running in a restricted shell, make sure the
+ pathname target of `hash -p' can be found via a $PATH search, to
+ prevent users assigning commands to the hash table they would not
+ ordinarily have access to.
+
+ 12/27
+ -----
+array.c,arrayfunc.c,...
+ - many changes to clean up unused variables and functions. From a
+ report from Siteshwar Vashisht <svashisht@redhat.com>
+
+ 12/28
+ -----
+lib/readline/terminal.c
+ - _rl_term_clrscroll: save the `E3' termcap capability, which clears
+ the scrollback buffer where supported. Unused right now
+
+lib/readline/rlprivate.h
+ - _rl_term_clrscroll: extern declaration
+
+lib/readline/bind.c
+ - parser_if: added simple variable comparison capability. Allowable
+ operators are `=', `==', and `!='; boolean variables must be
+ compared to either `on' or `off'; variable names must be separated
+ from the operator by whitespace
+
+doc/bash.1,lib/readline/doc/{rluser.texi,readline.3}
+ - document new `if variable comparison value' construct
+
+ 1/2/2018
+ --------
+lib/glob/sm_loop.c
+ - EXTMATCH: when matching against !(patlist), if a filename beginning
+ with a `.' does not match any of the patterns in patlist, don't
+ return it as a match if leading dots need to be matched explicitly
+ (flags & FNM_PERIOD). Report from Eric Cook <llua@gmx.com>
+
+ 1/3
+ ---
+variables.[ch]
+ - bind_function_def: takes an additional flags argument. If FLAGS&1,
+ overwrite any existing function_def hash table entry; if FLAGS==0,
+ leave any existing function_def alone.
+
+make_cmd.c
+ - make_function_def: call bind_function_def with flags == 0
+
+execute_cmd.c
+ - execute_intern_function: call bind_function_def with flags == 1 so
+ we have function_def information that's correct for where the
+ function is defined, not just where it's last parsed. Fixes report
+ from Bruno Vasselle <bruno.vasselle@laposte.net>; final piece of
+ fix from 12/15/2011
+
+ 1/4
+ ---
+subst.c
+ - param_expand: deal with string_list_dollar_star returning NULL.
+ Fixes bug reported by Martijn Dekker <martijn@inlv.org>
+
+builtins/history.def
+ - history_builtin: enabled code that performs range deletion
+
+ 1/5
+ ---
+subst.c
+ - do_assignment_internal: if performing a compound assignment, make
+ sure to pass ASS_CHKLOCAL flag to do_compound_assignment if the
+ assignment word has the W_CHKLOCAL flag set
+ - do_compound_assignment: honor ASS_CHKLOCAL flag and check for an
+ existing local variable before creating or modifying a global
+ variable
+
+builtins/declare.def
+ - declare_internal: new (undocumented so far) option: -G. Means to
+ act on global variables (create, modify) if no local variable is
+ found with the specified name
+ - declare_find_variable: new declare-specific wrapper functon for
+ declare builtin; obeys -g and -G options in one place
+ - declare_internal: if no variable is found after following any nameref
+ chain, look up the variable using declare_find_variable to honor the
+ -G option. XXX - so far, this is the only place that function is used
+
+subst.c
+ - shell_expand_word_list: before calling make_internal_declare, add
+ 'G' to the options list if W_CHKLOCAL is set in the word's flags.
+ This makes builtins like `readonly' that modify local variables in
+ a function behave the same for scalar and array variables
+
+ 1/11
+ ----
+parse.y
+ - shell_getc: move code that decides whether to append a space to an
+ alias expansion here from mk_alexpansion, so we can inhibit adding
+ a space if we're currently parsing a single or double quoted string
+
+ 1/12
+ ----
+
+parse.y
+ - clear_string_list_expander: take a pointer to an alias that's about
+ to be freed and make sure there aren't any pointers to it in the
+ list of pushed strings. If there are, zero it out in the pushed
+ string list to avoid referencing freed memory in pop_string()
+
+alias.c
+ - free_alias_data: if an alias being freed is currently being expanded,
+ call clear_string_list_expander to remove references to it from the
+ list of pushed strings
+
+ 1/14
+ ----
+pcomplib.c
+ - progcomp_search: add code to look up an alias for the CMD argument
+ and return the completions for the first word of that alias if one
+ is found. Just a start at completing aliases, a much-requested
+ feature
+
+pcomplete.h
+ - COPT_LASTUSER: last flag value used by user-settable completion
+ options
+ - PCOMP_RETRYFAIL, PCOMP_NOTFOUND: new #defines, possible return values
+ from programmable_completions in FOUNDP argument. Moved RETRYFAIL
+ define here from pcomplete.c to avoid collisions with user-settable
+ option values (COPT_*)
+
+ 1/15
+ ----
+pcomplete.c
+ - programmable_completions: if we don't find any completions for a
+ command, and RETRY is 0, see if the command is a defined alias,
+ expand it, and try to expand the first word of the value as a
+ command, and find any programmable completions for it. Here right
+ now, could be moved to attempt_shell_completion later if we need
+ to do more analysis of the expanded line. We'll see how it works
+ in practice. (Disabled for now.)
+
+ 1/16
+ ----
+parse.y
+ - grammar: when timing the null command, make sure to turn off the
+ flags in parser_state (PST_REDIRLIST) that make_simple_command sets
+ when given a NULL second argument, since it assumes that it's going
+ to turn those off when it gets the next word of the simple command
+ (which it never gets in this case). Fixes bug reported by
+ Anti Räis <antirais@gmail.com>
+
+ 1/19
+ ----
+lib/readline/rltty.c
+ - prepare_terminal_settings (termios/termio): if there is a function
+ bound to the VDISCARD character in the current keymap, set VDISCARD
+ to _POSIX_VDISABLE while readline is active. From a report from
+ Rhialto <rhialto@falu.nl>
+
+ 1/22
+ ----
+builtins/history.def
+ - histtime: check whether or not localtime() returns NULL, and make
+ sure we only call strftime() with a valid struct tm. This can happen
+ when the timestamps in the history file overflow a time_t. Fixes bug
+ reported by Luke Dashjr <luke@dashjr.org>
+
+bashline.c
+ - edit_and_execute_command: if we're in vi editing mode, make sure
+ we end up in insert mode after executing the commands from the
+ edited file. This seems to be what other shells do. Report from
+ Stan Marsh <gazelle@xmission.com>
+
+ 1/26
+ ----
+bashline.c
+ - command_word_completion_function: match alias and shell function
+ names case-insensitively if the readline completion-ignore-case
+ variable is set. Inspired by report from <odnehel@gmail.com>
+
+lib/readline/display.c
+ - update_line: when performing a dumb update after wrapping the line
+ (usually due to printing the prompt), make sure we adjust
+ _rl_last_c_pos if there are invisible characters in prompt lines
+ other than the first (we assume those invisible characters are in
+ the last line, which is nearly always the case). We adjust by the
+ total number of invisible chars less the number of invisible chars
+ in the first prompt line. From a report in
+ https://bugs.launchpad.net/ubuntu/+source/bash/+bug/1745273
+
+execute_cmd.c
+ - execute_command_internal: if redirections attached to a compound
+ command fail, make sure we discard the `internal_fifos' unwind-
+ protect frame after freeing the copied fifo_list and before returning
+
+ 1/30
+ ----
+lib/readline/rlprivate.h
+ - BRACK_PASTE_FINI: add \r to the end of the string to avoid problems
+ with the tty driver thinking the cursor was in the wrong position.
+ Fixes issue reported by Egmont Koblinger <egmont@gmail.com>
+
+ 1/31
+ ----
+lib/sh/zread.c
+ - zread,zreadintr: call check_signals() before calling read() to
+ minimize the race window between signal delivery, signal handling,
+ and a blocking read(2). Partial fix for FIFO read issue reported by
+ Oyvind Hvidsten <oyvind.hvidsten@dhampir.no>
+
+doc/{bash.1,bashref.texi}
+ - shopt: document `assoc_expand_once' shell option
+
+ 2/1
+ ---
+config-top.h
+ - DONT_REPORT_SIGTERM: define, so non-interactive shells will no
+ longer print termination messages for child processes killed by
+ SIGTERM
+
+ 2/6
+ ---
+lib/readline/text.c
+ - rl_insert: don't attempt to optimize typeahead if we are only reading
+ a fixed number of characters (rl_num_chars_to_read > 0)
+
+redir.c
+ - redir_special_open: if the shell is restricted, return a
+ RESTRICTED_REDIRECT error for attempts to open /dev/tcp and /dev/udp
+ sockets. Inspired by report from Blake Burkhart <bburky@bburky.com>
+ - do_redirection_internal: if redir_open returns RESTRICTED_REDIRECT,
+ return that right away (instead of errno) so we can print a better
+ error message
+
+ 2/11
+ ----
+jobs.c
+ - bgp_resize: fix problems with (pid_t) overflow when calculating new
+ size for table when js.c_childmax is near the limit of a pid_t
+ (e.g., 2**31 - 1 on a 32-bit system, or 2**63-1 on a system with
+ 32-bit pid_t). Fixes hang reported by Natanael Copa
+ <ncopa@alpinelinux.org> based on his patch from 12/8.
+ - bgp_resize: cap max table size for bgpids at MAX_CHILD_MAX (32768)
+
+include/typemax.h
+ - TYPE_MINIMUM, TYPE_MAXIMUM: updated definitions from coreutils-8.29,
+ silences some compiler warnings
+
+ 2/14
+ ----
+Makefile.in
+ - maybe-clean: use cd and pwd -P to test whether or not two directory
+ names identical, since topdir = '.' and BUILD_DIR = full pathname
+ when you use something like `bash ./configure'. Problem reported by
+ Michael Felt <aixtools@gmail.com>
+
+subst.c
+ - split_at_delims: if SD_NOQUOTEDELIM is in the flags argument, don't
+ treat `'' and `"' as candidates for possible sequences of delimiters,
+ even if they're part of the delimiter set (the delims argument).
+ Fixes problem with completing lines like `foo --bar='quux baz' xx'
+ reported by Nick Patavalis <npat@efault.net>
+
+ 2/15
+ ----
+copy_cmd.c
+ - copy_word_list: build the list in the right order, avoiding having
+ to reverse it at the end. Helps with long argument lists
+
+shell.c
+ - bind_args: build the argument list in the right order, avoiding
+ having to reverse it at the end.
+ - bind_args: only call push_args to save argc and argv as BASH_ARGC
+ and BASH_ARGV if debugging mode is enabled (debugging_mode != 0).
+ Inspired by report from Ambrose Feinstein <ambrose@google.com>
+ - bind_args: note that we've saved BASH_ARGC and BASH_ARGV by setting
+ bash_argv_initialized
+ - shell_reinitialize: reset bash_argv_initialized back to 0 so
+ BASH_ARGV and BASH_ARGC will be recreated if we're in debugging mode
+
+variables.c
+ - save_bash_argv: new function, initializes BASH_ARGV and BASH_ARGC
+ from the saved positional parameters
+ - init_bash_argv: initialize BASH_ARGV and BASH_ARGC if
+ bash_argv_initialized == 0
+
+builtins/shopt.def
+ - shopt_set_debug_mode: if we're turning on debug mode, initialize
+ BASH_ARGC and BASH_ARGV if bash_argv_initialized == 0
+
+ 2/16
+ ----
+execute_cmd.c
+ - execute_function: make sure BASH_ARGV and BASH_ARGC are initialized
+ before calling push_args (and before calling remember_args)
+
+builtins/source.def
+ - source_builtin: make sure BASH_ARGV and BASH_ARGC are initialized
+ before calling push_args (and before calling remember_args)
+
+builtins/evalfile.c
+ - _evalfile: if the shell compatibility level is 44 or lower, make
+ sure BASH_ARGV and BASH_ARGC are initialized before calling
+ array_push
+
+builtins/shopt.def
+ - compat44: new shell option. This will be the last compatXX option
+
+doc/{bash.1,bashref.texi}
+ - compat44: document new shell option
+
+ 2/21
+ ----
+builtins/common.h
+ - ISOPTION: check s[1] before s[2] to avoid out of bound reads. Fixes
+ bug reported by jeremy@feusi.co
+
+input.h
+ - B_SHAREDBUF: new flag for buffered input fds, indicates that this
+ fd shares its b_buffer with another fd
+
+input.c
+ - duplicate_buffered_stream: when duplicating *from* the buffer
+ corresponding to bash input, make sure the new buffer has B_SHAREDBUF
+ in the flags, because copy_buffered_stream copies pointers
+ - duplicate_buffered_stream: if the buffer being copied to has the
+ B_SHAREDBUF flag set, make sure the b_buffer doesn't get freed.
+ Fixes use-after-free bug reported by jeremy@feusi.co
+ - save_bash_input,close_buffered_stream: make sure a buffer with
+ B_SHAREDBUF set has its b_buffer set to NULL before calling
+ free_buffered_stream
+
+ 2/24
+ ----
+subst.c
+ - parameter_brace_expand_error: add parameter saying whether or not
+ we are checking whether value is null, so we can have different
+ error messages for ${x:?} and ${x?}. Report and fix from
+ don fong <dfong@dfong.com>
+
+ 3/5
+ ---
+lib/readline/bind.c
+ - _rl_read_file: instead of calling stat/open on the passed filename,
+ use open/fstat to avoid one possible filename translation and close
+ a small (benign) race condition. Report and fix from Roy Ivy
+ <roy.ivy.iii@gmail.com>
+
+ 3/11
+ ----
+variables.c
+ - makunbound: if new variable localvar_unset is non-zero, mark local
+ vars in previous scopes as invisible and unset so they will show
+ up as unset until that previous scope returns (similar to how local
+ variables in the current local scope are handled). localvar_unset
+ is currently set to 0 with no way for a script to change its value.
+ Eventually there will be an option to modify it. From a bug-bash
+ discussion started by Nikolai Kondrashov <spbnick@gmail.com> back
+ on 2/11/2018
+
+ 3/13
+ ----
+subst.c
+ - expand_word_list_internal: short-circuit and exit right away if a
+ variable assignment preceding a special builtin fails in posix mode
+ - expand_word_list_internal: if a variable assignment precedes an
+ empty command name (after expansion), fix to exit right away when
+ the shell is in posix mode
+
+ 3/15
+ ----
+doc/{bash.1,bashref.texi}
+ - Add text to Pathname Expansion clarifying that a slash must be
+ matched by a slash in the pattern when matching pathnames, but
+ not in other matching contexts. Suggested by <stormy1777@yahoo.com>
+
+builtins/common.c
+ - read_octal: allow octal numbers greater than 777 to accommodate
+ modes and umasks that include sticky/setuid/setgid bits. Report
+ and fix from Martijn Dekker <martijn@inlv.org>
+
+ 3/19
+ ----
+lib/readline/bind.c
+ - rl_generic_bind: make sure we only assign to prevkey in the loop
+ if the key sequence index is > 0, so ic is valid. Fixes bug
+ reported by Koichi Murase <myoga.murase@gmail.com>
+
+builtins/read.def
+ - read_builtin: be slightly less aggressive checking for timeouts and
+ SIGALRM: if we successfully read a character, don't check for a
+ timeout until we store or process it. Fixes timing problem
+ reported by Rob Foehl <rwf@loonybin.net>
+
+ 3/22
+ ----
+sig.c
+ - termsig_handler: add a call to exit(1) after the kill, just in case
+ there are circumstances where the SIG_DFL signal handler is
+ ignored. Report from Andrei Vagin <avagin@gmail.com>
+
+ 3/25
+ ----
+lib/readline/bind.c
+ - _rl_function_of_keyseq_internal: new internal function thta takes a
+ length parameter to accommodate NUL in the key sequence. Patch from
+ Koichi Murase <myoga.murase@gmail.com>
+ - rl_function_of_keyseq_len: new application-callable function that
+ takes a length parameter; otherwise equivalent to rl_function_of_keyseq.
+
+lib/readline/readline.h
+ - rl_function_of_keyseq_len: add extern declaration for new function
+
+lib/readline/doc/rltech.texi
+ - rl_function_of_keyseq_len: document new function interface
+
+bashline.c
+ - bash_execute_unix_command: use rl_function_of_keyseq_len to handle
+ key sequences with embedded NULs (\C-@). Fix from Koichi Murase
+ <myoga.murase@gmail.com>
+
+lib/readline/bind.c
+ - rl_bind_key_if_unbound,rl_bind_key_if_unbound_in_map: run the KEY
+ argument through rl_untranslate_keyseq to produce a symbolic sequence
+ that can encode \C-@.
+ - rl_bind_keyseq_if_unbound_in_map: translate the key sequence in order
+ to accommodate symbolic key sequences; should be a no-op for `raw'
+ key sequences such as the arrow key seqeunces from terminfo. Change
+ from Koichi Murase <myoga.murase@gmail.com>
+
+ 4/2
+ ---
+jobs.c
+ - wait_for: when setting the SIGINT signal handler to wait_sigint_handler
+ make sure we're not setting old_sigint_handler recursively, as we
+ can when running an external command in a trap we took after a
+ command exited due to SIGINT. We don't want to overwrite
+ old_sigint_handler here. Fixes bug reported by Dr. Werner Fink
+ <werner@suse.de>
+
+execute_cmd.c
+ - execute_disk_command: when there is a command_not_found_hook, make
+ sure the subshell turns off job control before running it, in case
+ it runs processes. We don't want it to manipulate process groups.
+ Fixes bug reported by ÐиÑиллов Ðима <dk.1997-fast@yandex.ru>
+ - execute_command_internal: make sure the command run by the `command'
+ builtin doesn't cause the ERR trap to be executed; wait for the
+ status to be returned by the command builtin. Fixes bug reported by
+ Martijn Dekker <martijn@inlv.org>
+
+ 4/4
+ ---
+subst.c
+ - process_substitute: handle longjmp back to top_level and function
+ returns (return_catch) in the child process, like command
+ substitution, so we don't longjmp back to some arbitrary spot from
+ the `exit' or `return' builtins, or on an expansion error, like
+ the command timing code. Fixes bug reported by Basin Ilya
+ <basinilya@gmail.com>
+
+ 4/6
+ ---
+parse.y
+ - read_token_word: when reading a matched pair of backquotes as part
+ of a word, treat it as quoted so the characters are read as a single
+ word, but do not let the presence of the backquote mark the word as
+ quoted. Fixes here-document delimiter bug reported by Denys Vlasenko
+ <dvlasenk@redhat.com>
+
+ 4/7
+ ---
+execute_cmd.c
+ - execute_case_command: call quote_string_for_globbing with the
+ QGLOB_CTLESC flag for both quoted and unquoted words, so it will
+ remove CTLESC/CTLESC in all cases while converting other quoted
+ characters to use a preceding backslash. Bug reported by
+ Martijn Dekker <martijn@inlv.org>
+
+ 4/9
+ ---
+smatch.c
+ - posix_cclass_only: helper function that checks whether a pattern has
+ only posix single-byte character classes ([:alpha:], etc.) or has
+ none at all
+ - xstrmatch: if running in a multibyte locale, make sure to short-
+ circuit to the single-byte matching code only if there are no
+ unrecognized character class names, since the wide character ctype
+ functions allow locales to define their own character class names
+ (e.g., "hyphen"). Fixes issue reported by yangyajing <yyj_cqu@163.com>
+
+ 4/10
+ ----
+configure.ac,cross-build/qnx.cache
+ - qnx: add a configure cache file for cross-building, treat qnx 7 like
+ qnx 6 in terms of cpp options. Fix from Brian Carnes
+ <bcarnes@google.com>
+
+aclocal.m4
+ - BASH_CHECK_DEV_STDIN: experimental change to test for /dev/stdin
+ independently of /dev/fd or /proc/self/fd. Suggested for QNX by
+ Brian Carnes <bcarnes@google.com>
+
+
+ 4/11
+ ----
+lib/glob/glob.c
+ - glob_testdir: return -2 if DIR is a symlink, to differentiate it from
+ any other kind of non-directory file
+ - glob_vector: if we have GX_ALLDIRS (globstar), we want to skip over
+ symlinks to directories, since we will pick up the real directory
+ later. Fixes incompatibility reported by Murukesh Mohanan
+ <murukesh.mohanan@gmail.com>
+
+bashline.c
+ - bash_execute_unix_command: changes to make READLINE_POINT apply to
+ characters instead of bytes when in a multibyte locale. Report and
+ fix from Koichi Murase <myoga.murase@gmail.com>
+
+ 4/12
+ ----
+builtins/evalstring.c
+ - parse_and_execute_cleanup: now takes an argument which is the value
+ of running_trap at some point before parse_and_execute was called;
+ changed callers in sig.c, builtins/evalfile.c
+
+builtins/common.h
+ - parse_and_execute_cleanup: changed prototype
+
+ 4/13
+ ----
+builtins/evalstring.c
+ - parse_and_execute_cleanup: if the argument holding the previous state
+ of running_trap is the same value as the current running_trap state,
+ don't call run_trap_cleanup: assume that there is a caller who will
+ take care of the cleanup after this returns. Fixes recursive trap
+ call on "eval return" reported by Martijn Dekker <martijn@inlv.org>
+
+parse.y
+ - read_a_line: if remove_quoted_newline is non-zero, indicating the
+ here-document delimiter is unquoted, we will be running the contents
+ of the here-document through word expansion and need to quote CTLESC
+ and CTLNUL in the input. Fixes bug with ^A in here document reported
+ by Jorge Alberto Baca Garcia <bacagarcia@me.com>
+
+ 4/18
+ ----
+pathexp.c
+ - quote_string_for_globbing: make sure the QGLOB_CTLESC code handles
+ both CTLESC CTLESC and CTLESC CTLNUL in the same way. Fixes bug
+ reported by Martijn Dekker <martijn@inlv.org>
+
+ 4/19
+ ----
+execute_cmd.c
+ - execute_command_internal: before executing any command in the current
+ shell, and before copying any existing FIFO list, call
+ reap_procsubs to unlink or close any process substitution pipes
+ associated with processes that have exited. Fixes hang in test suite
+ when trying to open a FIFO with no process having it open for
+ reading
+
+ 4/26
+ ----
+parse.y
+ - read_token_word: if returning REDIR_WORD for a {id}>foo construct,
+ for example, make sure to assign the_word to yylval.word before
+ returning, in case a recursive call to the parser overwrites it
+ (e.g., when evaluating array indexes). From a message to
+ austin-group-l from Stephane Chazelas <stephane.chazelas@gmail.com>
+
+lib/glob/sm_loop.c
+ - BRACKMATCH: if we have an invalid character class in an otherwise
+ well-formed bracket expression, don't try to match each character
+ of the (invalid) class individually; just skip over the class and
+ move on. From a message on the austin-group list from
+ Stephane Chazelas <stephane.chazelas@gmail.com>
+
+ 4/27
+ ----
+variables.c
+ - push_exported_var,push_func_var,push_temp_var: make sure to set the
+ context correctly in the variable we bind in the previous (non-temp)
+ scope. Report from Martijn Dekker <martijn@inlv.org>
+
+pathexp.c
+ - unquoted_glob_pattern_p: a pattern that contains a backslash can
+ have it removed by the matching engine (since backslash is special
+ in pattern matching), so if the pattern contains a backslash, and
+ does not end in a backslash, we need to return true. Fixes bug
+ reported by Robert Elz <kre@bmunnari.OZ.AU>
+
+lib/glob/glob_loop.c
+ - INTERNAL_GLOB_PATTERN_P: same change to return TRUE for a backslash
+ that doesn't end the pattern
+
+lib/sh/timeval.c
+ - print_timeval: use locale_decpoint() instead of fixed `.' to print
+ decimal point. Bug report in austin-group email from Joerg Schilling
+ <Joerg.Schilling@fokus.fraunhofer.de>
+
+lib/sh/clock.c
+ - print_clock_t: use locale_depoint() in the same way as print_timeval
+
+ 4/29
+ ----
+subst.c
+ - expand_cond_node: if special != 0, make sure to add QGLOB_CTLESC
+ to the flags passed to quote_string_for_globbing. Same issue as the
+ one with `case' fixed on 4/7, report from Martijn Dekker
+ <martijn@inlv.org>
+
+ 4/30
+ ----
+redir.c
+ - do_redirection_internal: r_close_this: if the file descriptor is
+ already closed before the shell is asked to close it, make sure to
+ add an undo list redirect to make sure it stays closed. Report from
+ Martijn Dekker <martijn@inlv.org>
+
+ 5/2
+ ---
+variables.c
+ - push_posix_temp_var: new function, takes the SHELL_VAR * passed as
+ an argument and uses the name and value to create a global variable
+ - merge_temporary_env: if posixly_correct is set, call
+ push_posix_temp_var to create global variables, otherwise call
+ push_temp_var to preserve the old behavior. Right now, it's only
+ called when in posix mode, but that might change. This undoes the
+ change from 4/27 when in posix mode
+
+ 5/3
+ ---
+sig.c
+ - struct that holds the terminating signal information has a new
+ field: whether that signal is expected to cause a core dump
+ - termsig_handler: if the call to kill(2) doesn't kill the process,
+ we have a problem. If our pid is not 1, we just exit with status
+ 128+sig (fake the sig exit status). If the pid is 1, we assume
+ we're in a Linux pid namespace and aren't allowed to send a signal
+ to ourselves. If we need to generate a core dump, we try to get
+ the kernel to SIGSEGV us by dereferencing location 0. If not, we
+ just exit with 128+sig. From a report and patch from Andrei Vagin
+ <avagin@virtuozzo.com>
+
+ 5/4
+ ---
+bashline.c
+ - bash_execute_unix_command: make sure that parse_and_execute is called
+ with newly-allocated memory to avoid prematurely freeing the
+ command. Report and fix from Koichi Murase <myoga.murase@gmail.com>
+
+ 5/7
+ ---
+builtins/shopt.def
+ - syslog_history: a shell option to control whether history is logged
+ to syslog; can be modified at runtime. Original patch from
+ Siteshwar Vashisht <svashisht@redhat.com>
+
+config-top.h
+ - SYSLOG_SHOPT: new configurable option, determines whether there is a
+ shell option to control syslogging history lines at runtime and sets
+ the default value of the option
+
+bashline.c
+ - syslog_history: new variable to control whether history lines are
+ sent to syslog; default value is the value of SYSLOG_SHOPT (or 1
+ if that's not defined)
+ - bash_add_history: send history lines to syslog if syslog_history is
+ non-zero
+
+ 5/10
+ ----
+
+variables.c
+ - push_var_context: if we are in Posix mode and manipulating the
+ temporary environment (temporary_env), implement behavior specified
+ in Posix interp 1009 and make sure that temporary assignments
+ preceding function calls modify the current environment *before*
+ the function is executed.
+
+[bash-5.0-alpha frozen]
+
+ 5/12
+ ----
+execute_cmd.c
+ - execute_in_subshell: subshells should set loop_level == 0, since
+ they are no longer "enclosed" by the loop, according to posix.
+ Report from Aeron.E. Wang <aeron.e.wang@gmail.com>
+
+ 5/24
+ ----
+Makefile.in
+ - pkgconfigdir: don't fail installing bash.pc if this directory doesn't
+ exist or isn't writable -- there's no error in failing to install
+ something nothing will use
+ - install-headers-dirs: creat $(pkgconfigdir) if it doesn't exist
+
+lib/sh/shmbchar.c
+ - utf8_mblen: replace with version from gnulib
+ - utf8_mbstrlen: reimplement using utf8_mblen so it handles invalid
+ multibyte sequences in the same way as mbstrlen
+
+lib/readline/mbutil.c
+ - _rl_utf8_mblen: utf-8 specific version of mblen from gnulib
+
+include/shmbutil.h
+ - ADVANCE_CHAR,COPY_CHAR_P: do better job detecting end of string in
+ UTF-8 locales (should not be called with an empty string, but to
+ be safe)
+
+ 5/25
+ ----
+lib/sh/utf8.c
+ - new file, utf-8-specific functions collected from other files
+
+externs.h
+ - extern declarations moved around for utf8.c
+
+include/shmbutil.h
+ - SADD_MBCHAR, SADD_MBQCHAR_BODY, ADVANCE_CHAR_P: if the locale is a
+ UTF-8 locale, don't bother with a call to mbrlen if the current
+ character cannot start a multibyte character
+
+variables.c
+ - push_var_context: only merge the temporary environment in posix mode
+ if we are executing a shell function (flags & VC_FUNCENV). Report
+ from Martijn Dekker <martijn@inlv.org>
+
+lib/readline/mbutil.c
+ - _rl_get_char_len: don't call mbrlen if we are in a UTF-8 locale and
+ the character cannot start a multibyte sequence
+
+builtins/read.def
+ - read_builtin: if mb_cur_max > 1, call read_mbchar only if we're not
+ in a utf-8 locale or, if we are, the character we just read indicates
+ the start of a multibyte sequence
+
+subst.c
+ - string_extract_verbatim: don't call MBRLEN if we're in a utf-8
+ locale and the current character can't start a multibyte sequence
+ - setifs: don't call MBRLEN if we're in a utf-8 locale and the first
+ character of $IFS can't start a multibyte sequence
+
+lib/readline/kill.c
+ - rl_bracketed_paste_begin: make sure we return 0 here if rl_insert_text
+ returns the right number of characters inserted to be consistent
+ with other functions. Returns 1 otherwise. Report and fix from
+ Gabe Krabbe <gabe@rtfs.de> back in March, 2018
+
+lib/readline/readline.c
+ - rl_subseq_result: make sure r is < 0 before checking map[ANYOTHERKEY]
+ to see if we shadowed a key that should now be tried
+
+ 5/27
+ ----
+variables.c
+ - assign_aliasvar: perform same validity check on subscript assignment
+ as alias builtin performs on name argument. Bug report from
+ Mike Jonkmans <bashbug@jonkmans.nl>
+
+ 5/29
+ ----
+builtins/setattr.def
+ - set_var_attribute: we should not propagate a variable assignment
+ preceding a builtin back to the calling environment unless the
+ shell is in posix mode. Since previous versions of the shell do
+ this, setting the shell compatibility level to 44 or less will
+ continue the propagation behavior
+
+ 6/1
+ ---
+lib/readline/histexpand.c
+ - history_tokenize_word: as part of teaching history tokenization more
+ and more about shell syntax, allow command and process subsitution
+ and extended globbing patterns to appear within a word being
+ tokenized and not just at the beginning. Fixes bug reported back in
+ 2/2017 by ecki@tofex.de
+
+bashhist.c
+ - load_history: use HISTSIZE_DEFAULT (still defaults to "500") to set
+ the initial value of $HISTSIZE. HISTSIZE_DEFAULT can be overridden
+ in config-top.h
+
+ 6/4
+ ---
+configure.ac
+ - make sure we link against an external readline library that's at
+ least version 8
+
+ 6/8
+ ---
+pcomplete.h
+ - INITIALWORD: internal compspec name for programmable completion on
+ the initial (usually the command) word
+
+bashline.c
+ - attempt_shell_completion: if we are in a command position and the
+ user has defined a compspec for INITIALWORD, use programmable
+ completion to complete command words. Original patch from
+ Luca Boccassi <bluca@debian.org>
+
+lib/readline/doc/rluser.texi,builtins/complete.def,doc/bash.1
+ - make it clearer that -D takes precedence over -E when supplied as
+ options to `complete', not when they are applied during completion
+
+builtins/complete.def
+ - complete_builtin,compgen_builtin: add support for -I option
+ - print_one_completion,print_compopts: display -I when appropriate
+
+lib/readline/doc/rluser.texi,doc/bash.1
+ - complete,compgen: document new -I option and its effect
+
+ 6/10
+ ----
+lib/readline/histfile.c
+ - read_history_range: don't apply the heuristic and try to append a
+ history line to an existing history entry if we don't have any
+ history entries. Bug and fix from Edward Huff <ejhuff@gmail.com>
+
+ 6/12
+ ----
+bashline.c
+ - attempt_shell_completion: don't all the programmable completion for
+ INITIALWORD if programmable completion is disabled
+ - attempt_shell_completion: make sure in_command_position remains set
+ for an empty command word on an otherwise blank line, making the
+ presence of assignment statements optional. Report from
+ Luca Boccassi <bluca@debian.org>
+
+ 6/20
+ ----
+lib/malloc/malloc.c
+ - morecore,internal_malloc,internal_free: requests for more than
+ 128K bytes (defined as MMAP_THRESHOLD and saved in the new
+ malloc_mmap_threshold variable) are now satisfied via mmap and
+ freed via munmap. We only use mmap if we have mmap and MAP_ANON
+ (or MAP_ANONYMOUS). These blocks are not available for splitting
+ or coalescing, so every request for 128K bytes or smaller is
+ satisfied via sbrk(). We don't use mremap for realloc yet, but
+ we could in the future
+
+lib/malloc/mstats.h
+ - malloc_stats: the malloc stats now include the number of calls to
+ mmap and the total number of bytes requested via mmap. The number
+ of calls to munmap is captured in each bucket's lesscore count
+
+lib/malloc/stats.c
+ - print_malloc_stats: now prints an indication of where the change from
+ sbrk to mmap takes place, and prints number of mmap calls and total
+ number of bytes allocated using mmap
+
+ 6/22
+ ----
+variables.c
+ - get_bashargcv: new dynamic "fetch" variable for BASH_ARGV and
+ BASH_ARGC, for backwards compatibility: if a script makes a
+ reference to either variable at the top level (not in a shell
+ function) without enabling debugging mode and not having initialized
+ the variable previously (using a simple semaphore), create the
+ variables
+
+ 6/25
+ ----
+configure.ac
+ - opt_bash_malloc: bash malloc no longer disabled for systems that
+ require eight-bit alignment; the bash malloc has had this for a
+ long time
+
+ 7/4
+ ---
+bashline.c
+ - pre_process_line: if command-oriented history is enabled, and the
+ line being expanded is the second or later in a multi-line command,
+ and we know the command is being saved as the current history entry,
+ decrease history_length before calling history_expand so references
+ like !! refer to the previous history entry as usual
+
+lib/readline/histexpand.c
+ - history_quoting_state: new variable, can be set by calling
+ application before calling history_expand to note that the string
+ being expanded is part of a quoted string. Can be set to a single
+ quote, a double quote, or 0 (no quoting)
+ - history_expand: look at history_quoting_state and honor the
+ single-quote setting by not expanding any initial portion of the
+ line before the closing single quote. This allows history expansions
+ to be performed on a line containing a closing single quote if they
+ appear after the single quote
+
+lib/readline/history.h
+ - history_quoting_state: extern declaration
+
+bashhist.c
+ - bash_history_inhibit_expansion: if history_quoting_state indicates
+ that this string is single-quoted, skip over the single-quoted
+ portion and determine whether or not the portion after the
+ quoted string needs to be inhibited from history expansion
+
+parse.y
+ - shell_getc: set history_quoting_state based on the contents of the
+ current delimiter before calling pre_process_line
+
+ 7/6
+ ---
+lib/readline/doc/hsuser.texi
+ - describe the default behavior of backslash and single and double
+ quotes
+
+lib/readline/doc/hstech.texi
+ - history_quoting_state: describe effect of setting this variable
+ - history_quotes_inihibit_expansion: expand the description to include
+ the default quoting behavior that setting this variable enables
+
+ 7/9
+ ---
+support/man2html.c
+ - unescape: use memmove instead of strcpy to handle overlapping strings
+ Report and fix from Bernhard M. Wiedemann <bwiedemann@suse.de>
+
+lib/sh/getenv.c
+ - getenv: check that environ is non-NULL before looking through it.
+ Report and fix from Keeley Hoek <keeley@hoek.io>
+
+ 7/12
+ ----
+braces.c
+ - mkseq: use better integer overflow handling for systems with 32-bit
+ ints and 64-bit intmax_ts. Bug reported by Simon Wörner
+ <mail@simon-woerner.de> as the result of fuzzing
+
+builtins/declare.def
+ - declare_internal: make sure bind_variable returns non-NULL when
+ setting attributes for a variable named as an argument to declare
+ that also appears in the temporary environment (and is a nameref).
+ Bug reported by Simon Wörner <mail@simon-woerner.de as the
+ result of fuzzing
+
+variables.c
+ - bind_variable_internal: if we're assigning through a nameref, don't
+ create a variable with an invalid name under any circumstances
+
+builtins/common.c
+ - get_job_spec: make sure to return NO_JOB if atoi() returns < 0 due
+ to integer overflow.
+ Bug reported by Simon Wörner <mail@simon-woerner.de as the
+ result of fuzzing
+
+ 7/13
+ ----
+execute_cmd.c
+ - execute_in_subshell: don't call set_sigint_handler if the subshell
+ is asynchronous, since it undoes the signal handler installed by
+ setup_async_signals. Fixes bug reported by Daniel Mills
+ <danielmills1@gmail.com>
+
+parse.y,externs.h
+ - reset_readahead_token: new convenience function for the rest of
+ the shell, resets token_to_read if it's a newline (as it will be
+ after reset_parser is called)
+
+eval.c
+ - reader_loop: if we're just going to execute one command, make sure
+ the read-ahead token isn't set to something that will result in a
+ NULL command (by calling reset_readahead_token), since the code
+ will take that as the one command and set EOF_Reached
+
+ 7/15
+ ----
+doc/{bash.1,bashref.texi}
+ - indirect expansion: clarify that the expansion works on parameters,
+ not just variables (NAMEs). Suggested by konsolebox
+ <konsolebox@gmail.com>
+
+ 7/16
+ ----
+doc/{bash.1,bashref.texi}
+ - INSIDE_EMACS: document its effect on line editing
+
+ 7/17
+ ----
+lib/readline/{readline.c,rlprivate.h}
+ - _rl_eof_found: new variable, private to the readline library, that
+ indicates whether the current call to readline() will return NULL
+ because we read EOF
+
+lib/readline/rltty.c
+ - rl_deprep_terminal: if bracketed paste mode is active, the last
+ character of the string to disable it is \r (to avoid confusing
+ the terminal driver about where the cursor is). In this case,
+ output a newline before returning so subsequent text (like the
+ `exit' bash prints) doesn't overwrite the prompt. Bug from
+ https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=903936
+
+variables.c
+ - make_local_assoc_variable: add second argument like corresponding
+ local array function, to allow this function to return an existing
+ local array variable to the caller for the caller to handle
+
+variables.h
+ - make_local_assoc_variable: change function prototype to add second
+ arg
+
+{subst.c,variables.c}
+ - make_local_assoc_variable: change callers
+
+builtins/declare.def
+ - declare_internal: call make_local_assoc_variable with a non-zero
+ second arg to have it return an existing local array variable to be
+ flagged as an error. Fixes bug reported by Grisha Levit
+ <grishalevit@gmail.com>
+ - declare_internal: call make_local_array_variable with unconditional
+ second argument of 1 for the same reason as above
+
+ 7/18
+ ----
+variables.c
+ - bind_invalid_envvar: new function, takes invalid names from the
+ initial environment (names that are not valid shell identifiers) and
+ stores them in a separate hash table (invalid_env)
+ - maybe_make_export_env: make sure to add names from invalid_env to
+ the export env
+ - assign_in_env: for now, prevent variable names that aren't shell
+ identifiers from being added to the temporary environment. Addresses
+ issue raised by Grisha Levit <grishalevit@gmail.com>
+
+test.c
+ - unary_test: rearrange code slightly to avoid a wasted variable lookup
+ if the argument to -v is a subscripted array reference
+
+ 7/19
+ ----
+variables.c
+ - nameref_transform_name: if a name doesn't resolve to a shell variable,
+ this function will check whether it resolves to a nameref that
+ points to a variable that hasn't been created yet
+
+variables.h
+ - nameref_transform_name: extern declaration
+
+subst.c
+ - do_compound_assignment: make sure that we follow any nameref chain
+ if the name passed resolves to a nameref that points to a variable
+ that doesn't exist. Fixes issue raised by Grisha Levit
+ <grishalevit@gmail.com>
+
+builtins/declare.def
+ - declare_internal: before calling any variant of make_local_variable,
+ make sure to perform any transformation of the name indicated by an
+ existing nameref. Fixes issue raised by Grisha Levit
+ <grishalevit@gmail.com>
+
+ 7/20
+ ----
+builtins/declare.def
+ - declare_internal: if we are creating a global variable with -g, even
+ if we're not giving it a value, check for namerefs at the global
+ scope to avoid confusion with namerefs at the local (function) scope.
+
+subst.c
+ - expand_word_internal: if a double-quoted string expands to nothing,
+ make sure we note that for later by setting had_quoted_null, just
+ as we do for single-quoted empty strings
+
+subst.[ch]
+ - W_SAWQUOTEDNULL: new flag (replaces W_HASCTLESC, which is unused),
+ means that we saw a possibly-discarded quoted null while expanding
+ this word
+
+subst.c
+ - expand_word_internal: if expansion results in a non-empty word but
+ we saw a quoted null during expansion (had_quoted_null == 1), set
+ W_SAWQUOTED_NULL in the returned word
+ - expand_word_internal: if a recursive call to param_expand comes back
+ with W_SAWQUOTEDNULL set in the resulting word, set had_quoted_null
+ to note it
+ - parameter_brace_expand_rhs: if a recursive call to expand_word_internal
+ returns a non-quoted-null string (after an optional call to
+ string_list) make sure we pass the W_SAWQUOTEDNULL flag back to the
+ caller
+ - word_list_split: if a word expands to nothing after expansion and
+ splitting, but we saw a quoted null during the expansion
+ (W_SAWQUOTEDNULL), return an empty word
+
+ 7/25
+ ----
+
+subst.c
+ - do_compound_assignment: if creating a local variable, make sure to
+ set `newname' to the name of the variable returned from find_variable,
+ since that follows namerefs. Fixes issue raised by Grisha Levit
+ <grishalevit@gmail.com>
+
+ 7/29
+ ----
+subst.c
+ - get_var_and_type: if VALUE is NULL, check before calling dequote_string.
+ Report and fix from Grisha Levit <grishalevit@gmail.com>
+
+ 7/30
+ ----
+variables.c
+ - make_local_{array,assoc}_variable: make sure we're not trying to
+ inherit a value from an incompatible array type. Fixes issue raised
+ by Grisha Levit <grishalevit@gmail.com>
+ - nameref_transform_name: if we're trying to resolve a nameref that
+ will be used to create a local variable, make sure the nameref is
+ at the same variable scope. Report from Grisha Levit
+ <grishalevit@gmail.com>
+
+ 8/2
+ ---
+array.c
+ - array_subrange: change to use string_list_pos_params after creating a
+ WORD_LIST from the array slice, like assoc_subrange does
+
+subst.c
+ - parameter_brace_substring: since assoc_subrange and array_subrange
+ both call string_list_pos_params now, treat the results the same as
+ the VT_POSPARAMS case (pos_params also calls string_list_pos_params).
+ Fixes behavior difference between ${a[@]:sub} and ${@:sub} reported
+ by Ilkka Virta <itvirta@iki.fi>
+
+ 8/3
+ ---
+array.c
+ - array_patsub: rewrite to work in terms of a WORD_LIST * and call
+ string_list_pos_params on the result to be consistent with the
+ expansions of ${@/pat/rep} and ${*/pat/rep}
+
+assoc.c
+ - assoc_patsub: rewrite to work in terms of a WORD_LIST * and call
+ string_list_pos_params on the result to be consistent with the
+ expansions of ${@/pat/rep} and ${*/pat/rep}
+
+subst.c
+ - parameter_brace_patsub: change how return value of {array,assoc}_patsub
+ is treated to make it identical to pos_params_pat_subst, since they
+ all call string_list_pos_params now
+ - expand_string_for_pat: make sure we preserve the value of
+ expand_no_split_dollar_star instead of just unconditionally setting
+ it back to 0 in case it was 1 before this function was called
+
+ 8/6
+ ---
+array.c
+ - array_modcase: rewrite to work in terms of a WORD_LIST * and call
+ string_list_pos_params on the result to be consistent with the
+ expansions of ${@,,} and ${*,,}
+
+assoc.c
+ - assoc_modcase: rewrite to work in terms of a WORD_LIST * and call
+ string_list_pos_params on the result to be consistent with the
+ expansions of ${@,,} and ${*,,}
+
+subst.c
+ - parameter_brace_casemod: change how return value of {array,assoc}_modcase
+ is treated to make it identical to pos_params_modcase, since they
+ all call string_list_pos_params now
+
+ 8/8
+ ---
+builtins/declare.def
+ - declare_internal: if we are making local variables, and not dealing
+ with the nameref attribute, make sure that any nameref variable we
+ followed when resolving the name given was at the same variable
+ context. If not, we just want to make or use a local variable with
+ the name passed; if so, we want to use the nameref value as the
+ variable name. Report from Grisha Levit <grishalevit@gmail.com>
+
+ 8/9
+ ---
+configure.ac
+ - globasciiranges: RRI now on by default, must be turned off explicitly
+ at configure time or runtime with `shopt -u globasciiranges'
+
+ 8/14
+ ----
+variables.c
+ - dispose_saved_dollar_vars: decrement stack pointer before looking
+ for saved positional parameters to dispose; stack pointer always
+ points to the first unused slot
+
+ 8/15
+ ----
+variables.c
+ - dollar_arg_stack: now a stack of struct saved_dollar_vars, which has
+ an array for the first ten (dollar_vars) and a WORD_LIST * for the
+ remaining (rest_of_args). Fixes performance issue with function calls
+ and large numbers of positional parameters raised by
+ Bize Ma <binaryzebra@gmail.com>
+ - {save,restore,free,free_saved}_dollar_vars: new functions to manage
+ dollar_vars and dollar_arg_stack members. Need to keep these in sync
+ with whatever remember_args does
+ - push_dollar_vars: use save_dollar_vars, which just copies pointers,
+ and directly assign rest_of_args, without copying the words, to the
+ dollar_arg_stack entry. Have to clear dollar_vars and rest_of_args
+ with the assumption that callers will call remember_args(args, 1)
+ immediately following
+ - pop_dollar_vars: free current positional parameters and restore old
+ ones from pointers saved in dollar_arg_stack, making sure to
+ invalidate any cached value for "$@"
+ - dispose_saved_dollar_vars: free saved pointers from current index
+ into dollar_arg_stack
+
+doc/{bash.1,bashref.texi}
+ - POSIXLY_CORRECT: make sure to note that bash makes sure this variable
+ is set when posix mode is enabled
+
+ 8/17
+ ----
+{jobs,nojobs}.c
+ - set_jobs_list_frozen: set jobs_list_frozen to a particular value.
+ Intended to save and restore the value around code sections instead
+ of unconditionally unfreezing it.
+
+jobs.h
+ - set_jobs_list_frozen: extern declaration
+
+execute_cmd.c
+ - execute_pipeline: if lastpipe is enabled, save and restore the
+ value of jobs_list_frozen using freeze_jobs_list/set_jobs_list_frozen
+ to avoid problems with race conditions and nested pipelines
+ causing jobs to be removed from the jobs table. Fixes savannah issue
+ https://savannah.gnu.org/support/index.php?109541 reported by
+ Björn Kautler <vampire0>
+
+ 8/24
+ ----
+execute_cmd.c
+ - lastpipe_cleanup: call set_jobs_list_frozen instead of
+ unfreeze_jobs_list
+ - execute_pipeline: set up lastpipe_cleanup with old value of
+ jobs_list_frozen
+
+ 9/3
+ ---
+builtins/printf.def
+ - getuintmax,getfloatmax: on a conversion error, return as much of the
+ value as we were able to convert instead of 0. Fixes bug reported
+ by Robert Elz <kre@bmunnari.OZ.AU>
+
+ 9/4
+ ---
+lib/readline/text.c,lib/readline/rlprivate.h
+ - _rl_backward_char_internal: new function, guts of rl_backward_char
+ and rl_backward_byte, not currently used there
+
+lib/readline/vi_mode.c
+ - _rl_vi_advance_point: new function, move point forward by one
+ character, handling multibyte locales and characters and the end
+ of line semantics
+ - _rl_vi_backup_point: new function, move point backward by one
+ character, handling multibyte locales and characters
+ - rl_vi_eword,rl_vi_eWord: use rl_vi_advance_point instead of a simple
+ increment to handle multibyte characters. Fixes bug reported by
+ Enrico Maria De Angelis <enricomaria.dean6elis@gmail.com>
+
+ 9/5
+ ---
+lib/readline/vi_mode.c
+ - rl_vi_fword,rl_vi_fWord: use rl_vi_advance_point instead of a simple
+ increment to handle multibyte characters
+ - rl_vi_bword,rl_vi_bWord: use rl_vi_backup_point instead of a simple
+ decrement (and _rl_vi_advance_point where necessary) to handle
+ multibyte characters
+ - rl_vi_complete,_rl_vi_change_mbchar_case,_rl_vi_domove_motion_cleanup:
+ use _rl_vi_advance_point instead of simple rl_point increment
+ - vi_delete_dispatch,vi_change_dispatch,vi_yank_dispatch: use
+ INCREMENT_POS instead of a simple increment to rl_mark to handle
+ multibyte characters
+ - rl_vi_column: use _rl_forward_char_internal, starting with
+ rl_point == 0, to handle multibyte characters (Posix says `character
+ position', not index)
+
+ 9/7
+ ---
+configure.ac
+ - changed release status to `beta'
+
+ 9/9
+ ---
+lib/readline/display.c
+ - _rl_update_final: if the bottom line has zero characters and we are
+ on that line at column 0, don't bother with an additional \r\n.
+ Fixes redisplay nit reported by Per Bothner <per@bothner.com>
+
+
+configure.ac
+ - openbsd needs DEV_FD_STAT_BROKEN defined
+
+[bash-5.0-beta frozen]
+
+ 9/11
+ ----
+builtins/exec.def
+ - exec_builtin: make sure to sync the buffered stream where bash is
+ reading input (especially if it's fd 0) so a command exec'd by the
+ script can read the rest of stdin after the exec
+
+ 9/15
+ ----
+lib/readline/histexpand.c
+ - history_tokenize_internal: if the event contains embedded newlines
+ (e.g., bash with command-oriented history and lithist), use them as
+ word delimiters, equivalent to space and tab, so they don't end up
+ as separate words. Fixes issue pointed out by Viktor Dukhovni
+ <ietf-dane@dukhovni.org>
+ - history_tokenize_word: don't break if we get a newline (though we
+ shouldn't get one due to the loop in history_tokenize_internal
+ - history_expand_internal: use newline as a whitespace character when
+ expanding by words, as we do with history_tokenize_internal
+
+jobs.h
+ - J_PIPEFAIL: new flag for `flags' element of job struct
+
+jobs.c
+ - stop_pipeline: if pipefail_opt set, newjob gets J_PIPEFAIL in its
+ flags word
+ - raw_job_exit_status: use J_PIPEFAIL (setting of pipefail when job
+ created) instead of current setting of pipefail status to determine
+ how to compute exit status of pipeline. Tentative implementation of
+ Posix proposal
+
+expr.c
+ - exp0: don't call expr_bind_variable with a NULL string. Fixes
+ fuzzing bug reported by Eduardo Bustamante <dualbus@gmail.com>
+ - expr_bind_variable: don't try to do anything with a NULL or empty
+ LHS
+
+ 9/16
+ ----
+lib/readline/undo.c
+ - rl_do_undo: before we release the undo list entry we've just
+ processed, make sure we avoid any pointer aliasing issues caused
+ by having the entry being removed as part of the undo list in
+ _rl_saved_line_for_history. Fixes fuzzing bug reported by
+ Eduardo Bustamante <dualbus@gmail.com>
+
+ 9/17
+ ----
+[bash-5.0-beta released]
+
+ 9/18
+ ----
+lib/readline/bind.c
+ - name_and_keymap: new struct for keymap names and maps
+ - builtin_keymap_names: static array of builtin keymap names and
+ maps; preparing for allowing applications to set the names of
+ keymaps they create; keymap_names is initially a pointer to
+ this array
+ - _rl_get_keymap_by_name,_rl_get_keymap_by_map: new functions for
+ searching the keymap_names array and returning an index
+ - rl_get_keymap_by_name, rl_get_keymap_name: rewritten in terms of
+ new functions above
+ - rl_set_keymap_name (char *name, Keymap map): new function, set
+ name of MAP to NAME. NAME must not be builtin; MAP must not be one
+ of the builtin keymaps. Request and initial implementation from
+ Tom Tromey <tom@tromey.com>
+
+lib/readline/readline.h
+ - rl_set_keymap_name: new extern declaration for new public function
+
+lib/readline/doc/rltech.texi
+ - rl_set_keymap_name: add documentation
+
+lib/readline/doc/rluser.texi
+ - add text to `set keymap' description to note that applications
+ can add keymap names that can be used there
+
+ 9/20
+ ----
+parse.y
+ - shell_getc: don't execute the alias hack (returning a space at the
+ end of the string) if we are parsing a command substitution that
+ starts with a double paren (subshell inside a comsub), in which
+ case the flags are PSH_DPAREN. Fixes fuzzing bug reported by
+ Eduardo Bustamante <dualbus@gmail.com>
+
+lib/readline/isearch.c
+ - _rl_isearch_dispatch: default case: make sure we check multibyte
+ char length when deciding whether to enlarge the search string
+ buffer, instead of using the old assumption. Fixes fuzzing bug
+ reported by Eduardo Bustamante <dualbus@gmail.com>
+
+builtins/fc.def,execute_cmd.c
+ - fixed some missing free()s uncovered by coverity. Report from
+ Siteshwar Vashisht <svashisht@redhat.com>
+
+lib/glob/glob.c
+ - glob_vector: make sure name_vector is initialized to NULL
+
+lib/sh/{pathcanon,pathphys}.c
+ - {pathcanon,pathphys}: use memmove instead of strcpy on a possibly-
+ overlapping region of memory
+
+subst.c
+ - parameter_list_transform: make sure to dispose the word list in all
+ cases before returning
+ - parameter_brace_expand_rhs: make sure t1 is freed before returning
+ due to an invalid name resulting from an indirect expansion
+
+support/man2html.c
+ - fixed a couple of memory leaks
+
+ 9/21
+ ----
+subst.c
+ - process_substitute: if we are part of a job control process chain
+ (pipeline_pgrp != shell_pgrp), have the child shell forked to run
+ the process substitution set pipeline_pgrp to its own PID,
+ effectively becoming a process group leader without changing
+ its own process group. Fixes stray SIGHUP issue reported by
+ Jeremy Townshend <jeremy.townshend@gmail.com>
+
+ 9/23
+ ----
+arrayfunc.c
+ - assign_array_element: if we are assigning to an existing associative
+ array, and assoc_expand_once is set, allow `*' and `@' as subscripts.
+ Partial fix for report from Grisha Levit <grishalevit@gmail.com>
+
+variables.c
+ - bind_int_variable: if valid_array_reference (lhs) is not true,
+ make sure that the lhs is a valid identifier before assigning the
+ value
+
+arrayfunc.c
+ - valid_array_reference: allow blank subscripts. They are treated as
+ `normal' keys for associative arrays and evaluate to 0 for indexed
+ arrays. More of fix for report from Grisha Levit
+ <grishalevit@gmail.com>
+
+ 9/24
+ ----
+bashline.c
+ - restore_tilde: if the expanded length (xl) is longer than vl
+ we end up requesting a negative amount of memory (an extremely
+ large unsigned number). Just punt and return val in this case.
+ Fuzzing bug reported by Eduardo Bustamante <dualbus@gmail.com>
+ - restore_tilde: make sure we return what the user typed if tilde
+ expansion fails
+
+ 9/29
+ ----
+builtins/shopt.def
+ - uncomment `localvar_unset' option definition
+
+doc/{bash.1,bashref.texi}
+ - document `localvar_unset' shell option
+
+arrayfunc.c
+ - valid_array_reference: if we are parsing a subscript for an existing
+ associative array, the `assoc_expand_once' option is set, and the
+ VA_ONEWORD flag is set in FLAGS (meaning there should be nothing
+ following the closing `]'), don't call skipsubscript to find the
+ closing `]', use one that is at the end of the word. Part of fix for
+ issue reported by Grisha Levit <grishalevit@gmail.com>
+
+builtins/{printf,set}.def
+ - pass VA_ONEWORD as part of flags value everywhere valid_array_reference
+ is used
+
+config-top.h
+ - CHECKWINSIZE_DEFAULT: now 1, so check_window_size is on by default
+ - HISTEXPAND_DEFAULT: new define, allows builder to enable or disable
+ history expansion by default at build time
+
+doc/{bash.1,bashref.texi}
+ - checkwinsize: document new default value
+
+bashhist.h
+ - HISTEXPAND_DEFAULT: don't define if it's already defined. Strict
+ POSIX mode continues to default to off
+
+ 9/30
+ ----
+lib/readline/input.c
+ - win32_isatty: win32-specific changes from GDB. Patch submitted by
+ Tom Tromey <tom@tromey.com>, originally from Eli Zaretskii
+ <eliz@gnu.org>
+
+ 10/1
+ ----
+lib/readline/vi_mode.c
+ - rl_vi_start_inserting: start an undo group so we can grab the text
+ inserted here as a single unit, avoiding any insert coalescing
+ performed by rl_insert_text. Report and fix from Richard Todd
+ <richard.w.todd@outlook.com>
+
+ 10/3
+ ----
+subst.c
+ - parameter_brace_transform: if we're asked to display the attributes
+ of an unset variable, check that the variable exists even if
+ get_var_and_type returns NULL (it checks invisible_p). Requested by
+ Michal Pesa <pesathem@gmail.com>
+
+lib/readline/kill.c
+ - _rl_bracketed_text: new function, collects the text pasted in
+ bracketed paste mode and consumes the bracketed paste end marker;
+ returns the pasted text and its length
+ - rl_bracketed_paste_begin: call _rl_bracketed_text to collect the
+ text
+
+lib/readline/isearch.c
+ - _rl_isearch_dispatch: set cxt->lastc to -7 if the input resolves to
+ rl_bracketed_paste_begin
+ - _rl_isearch_dispatch: if cxt->lastc == -7, call _rl_bracketed_text
+ to collect the pasted text and add it to the accumulating search
+ string. Only works if ESC is not one of the isearch terminators.
+ Fixes issue reported in Debian bug report 891780,
+ https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=891780
+
+ 10/5
+ ----
+variables.c
+ - push_posix_tempvar_internal: new function, body of push_func_var
+ with additional argument saying whether it's being called from a
+ function or (special) builtin code path
+ - push_builtin_var: new function, calls push_posix_tempvar_internal
+ with second argument indicating builtin code path
+ - push_func_var: now a stub that calls push_posix_tempvar_internal
+ with second argument indicating function code path
+ - pop_scope: if called from a special builtin, call push_builtin_var
+ instead of push_func_var to do the right variable propagation
+
+builtins/shopt.def
+ - progcomp_alias: uncomment, make available to users
+
+doc/{bash.1,bashref.texi},lib/readline/doc/rluser.texi
+ - progcomp_alias: document shopt option, describe its use in
+ the section on programmable completion
+
+trap.c
+ - decode_signal: handle SIGRTMIN+n at runtime, with the `SIG' prefix
+ or without, case-insensitively if requested, and return SIGRTMIN+n.
+ These values could be different than what was available at compile
+ time. Report and patch from Rasmus Villemoes <rv@rasumsvillemoes.dk>
+
+ 10/8
+ ----
+execute_cmd.c
+ - execute_command_internal: only set line_number from command->value.Subshell
+ if the type == cm_subshell; otherwise defer and set later
+
+ 10/10
+ -----
+examples/loadables/seq.c
+ - seq: new loadable builtin, derived originally from coreutils:seq.c
+ but with very little of that code remaining
+
+ 10/12
+ -----
+trap.c
+ - run_pending_traps,_run_trap_internal: honor evalnest_max and
+ increment/decrement evalnest accordingly, since trap actions
+ are processed as if run by `eval'. Feature suggsted by Mike
+ Gerwitz <mtg@gnu.org>
+
+ 10/16
+ -----
+expr.c
+ - expr_skipsubscript: new function, calls skipsubscript with flags
+ similar to arrayfunc.c:valid_array_subscript if assoc_expand_once
+ is set and it looks like we've already expanded the subscript of
+ an associative array. Reported back on 8/27 by Grisha Levit
+ <grishalevit@gmail.com>
+ - readtok: call expr_skipsubscript instead of skipsubscript
+
+arrayfunc.c
+ - valid_array_reference: call skipsubscript with a third arg computed
+ from the VA_NOEXPAND flag only if we're expanding an associative
+ array subscript -- we already figure out whether or not we are
+
+[bumped release status to beta2]
+
+ 10/20
+ -----
+builtins/setattr.def
+ - set_or_show_attributes: after isolating NAME, make sure to restore
+ the "[+]=" in case we need the word later. Issue pointed out by
+ Grisha Levit <grishalevit@gmail.com>
+
+ 10/21
+ -----
+lib/readline/search.c
+ - noninc_search_from_pos: if we are supposed to be searching for a
+ pattern (vi mode), make sure to pass S to _hs_history_patsearch,
+ since that has any leading `^' stripped
+
+lib/readline/histsearch.c
+ - _hs_history_patsearch: if the search isn't anchored, put a `*' at
+ the beginning to force fnmatch to match anywhere in the line (could
+ look at this later to make a change to history_search_internal that
+ would avoid the need to add the leading `*')
+
+subst.c
+ - parameter_brace_expand_rhs: treat a failure to assign a variable with
+ a ${param:=value} expansion as an expansion error, and, in a non-
+ interactive posix-mode shell, exit the shell
+ - param_expand: don't set W_SPLITSPACE for $* unless IFS is NULL;
+ consistent with other uses of W_SPLITSPACE
+
+ 10/22
+ -----
+doc/{bash.1,bashref.texi}
+ - tweak description of bash conditional expressions to note that the
+ test and [ commands determine their behavior based on the number of
+ arguments. Suggested by Ilkka Virta <itvirta@iki.fi>
+
+ 10/24
+ -----
+execute_cmd.c
+ - execute_simple_command: don't set $_ to NULL when executing a
+ command that forks; just leave it unchanged. Tweaked the documentation
+ slightly as a result. Inspired by report from Ricky Tigg
+ <ricky.tigg@gmail.com>
+
+ 10/28
+ -----
+redir.c
+ - here_document_to_fd: make sure the temp files used to store here
+ documents are readable (and writable, where necessary) by the user.
+ This can happen in the unlikely case that someone decides to shoot
+ himself in the foot by setting the umask to 400. Issue originally
+ raised back in March by Stephane Chazelas
+ <stephane.chazelas@gmail.com>; fix inspired by Martijn Dekker
+ <martijn@inlv.org>
+
+ 10/29
+ -----
+lib/readline/terminal.c
+ - bind_termcap_arrow_keys: bind the "kI" capability (what the Insert
+ keypad key outputs) to overwrite mode. Patch from Xose Vazquez Perez
+ <xose.vazquez@gmail.com>
+
+ 10/30
+ -----
+braces.c
+ - mkseq: make sure to terminate result array before passing it to
+ strvec_dispose on an interrupt. Report and fix from Corbin Souffrant
+ <corbin.souffrant@gmail.com>
+
+ 11/6
+ ----
+trap.c
+ - _run_trap_internal: if the signal shows up as SIG_CHANGED after
+ running the trap handler, check for terminating signals and run any
+ terminating signal handler indicates. Fixes issue reported by
+ Owen Stephens <owen@owenstephens.co.uk>
+
+ 11/7
+ ----
+execute_cmd.c
+ - execute_builtin: don't merge the temporary environment when the
+ `return' builtin is being executed if it's being executed by the
+ `command' builtin, since that's supposed to inhibit the special
+ builtin properties. Part of POSIX conformance problems reported
+ by Martin Rehak <martin.rehak@oracle.com>
+
+builtins/cd.def
+ - bindpwd: if canonicalization fails when -P is specified (e.g. if the
+ directory name length exceeds PATH_MAX), reset dirname ($PWD value) =
+ to the_current_working_directory (Posix cd description, step 10.)
+ Part of POSIX conformance problems reported by Martin Rehak
+ <martin.rehak@oracle.com>
+
+builtins/kill.def
+ - kill_builtin: support -sSIG and -nSIG without requiring them to be
+ separate arguments. POSIX says a "conforming implementation" should
+ accept them.
+ Part of POSIX conformance problems reported by Martin Rehak
+ <martin.rehak@oracle.com>
+
+ 11/8
+ ----
+arrayfunc.c
+ - array_value_internal: return NULL for invisible array variables.
+ Fixes issue with FUNCNAME sometimes returning invalid value reported
+ by Great Big Dot <greatbigdot@gmail.com>
+
+builtins/cd.def
+ - change_to_directory: if we are in posix mode, and the chdir to the
+ absolute pathname ($PWD/dirname) fails, return an error without
+ trying to use just `dirname' (posix cd step 10).
+ Part of POSIX conformance problems reported by Martin Rehak
+ <martin.rehak@oracle.com>
+
+execute_cmd.c
+ - execute_case_command: use expand_word_leave_quoted to expand the
+ word, then dequote the resulting string. This performs the
+ expansions in the order posix specifies.
+ Part of POSIX conformance problems reported by Martin Rehak
+ <martin.rehak@oracle.com>
+
+subst.c
+ - expand_word_unsplit: rewrite in terms of expand_word_leave_quoted
+ - ifs_whitespace: new macro, Posix ifs whitespace (ISSPACE; member
+ of current locale's space char class)
+ - ifs_whitesep: new macro, whitespace that is a member of $IFS; used
+ by list_string and get_word_from_string
+ - list_string: use ifs_whitesep instead of spctabnl in case $IFS
+ contains whitespace characters that are not space, tab, or newline
+ - get_word_from_string: use ifs_whitesep instead of spctabnl in case
+ $IFS contains whitespace characters that are not space, tab, or
+ newline.
+ Part of POSIX conformance problems reported by Martin Rehak
+ <martin.rehak@oracle.com>
+
+ 11/9
+ ----
+general.c
+ - posix_initialize: enable the shift_verbose option when turning on
+ posix mode, and disable it when posix mode is disabled. From a
+ report by Eric Blake <eblake@redhat.com>
+
+doc/bashref.texi
+ - posix mode: note the effect of posix mode on shift_verbose
+
+ 11/12
+ -----
+subst.c
+ - parameter_brace_expand: if parameter_brace_expand_word returns an
+ error, make sure to set TEMP = 0 (to note that the variable is unset).
+ From a report by Grisha Levit <grishalevit@gmail.com>
+ - param_expand: if expanding $! when set -u is enabled, honor the
+ PF_IGNUNBOUND flag and just return NULL, relying on the caller to
+ take care of understanding that the variable is unset
+ From a report by Grisha Levit <grishalevit@gmail.com>
+ - parameter_brace_expand: if we are checking nullness, and we have a
+ valid array expansion, a quoted null string resulting from the array
+ expansion of a * or @ subscript satisfies the nullness check.
+ From a report by Grisha Levit <grishalevit@gmail.com>
+
+ 11/14
+ -----
+aclocal.m4
+ - BASH_CHECK_LIB_TERMCAP: add last-ditch check for libncursesw, since
+ it's apparently the only curses library on some distributions.
+ Report from John Frankish <john.frankish@outlook.com>
+
+ 11/16
+ -----
+bashline.c
+ - attempt_shell_completion: allow an initial word completion to
+ complete a partial first word (point appears in the middle of
+ the word) before resorting to command completion. Report and fix
+ from Luca Boccassi <bluca@debian.org>
+
+[bash-5.0-beta2 frozen]
+
+ 11/27
+ -----
+bashline.c
+ - attempt_shell_completion: better fix for problems with fix from
+ 11/16. Report and fix from Tom Ryder <tom@sanctum.geek.nz>
+
+expr.c
+ - expr_skipsubscript: fix return type. Report and fix from
+ Andreas Schwab <schwab@linux-m68k.org>
+
+ 11/29
+ -----
+subst.c
+ - quote_escapes_internal: refactored quote_escapes into a function
+ that takes an additional flag saying whether or not we are going
+ to split the result. If we are not, and CTLESC is in IFS, we quote
+ the CTLESC (ditto for CTLNUL) to prevent it being removed when the
+ string is dequoted.
+ - quote_escapes: call quote_escapes_internal with FLAGS == 0
+ - quote_rhs: new function, calls quote_escapes with FLAGS == PF_NOSPLIT2
+ so that a CTLESC will be quoted if CTLESC is in $IFS
+ - parameter_brace_expand_word: call quote_rhs on the value of a variable
+ if PFLAGS includes PF_ASSIGNRHS, indicating that we will not be
+ splitting the word, but we will be dequoting it. Fixes bug reported
+ by Martijn Dekker <martijn@inlv.org>
+ - param_expand: same change as for parameter_brace_expand_word
+
+execute_cmd.c
+ - execute_in_subshell: if we are running a trap (running_trap > 0),
+ turn off the SIG_INPROGRESS and SIG_CHANGED flags for that signal
+ by calling run_trap_cleanup and reset running_trap to 0 (watch the
+ second part!). Tagged for bash-5.1.
+
+ 11/30
+ -----
+lib/readline/doc/rltech.texi
+ - rl_set_keymap_name: correct typo in the name; some updates to the
+ description that clarify usage. Report from <hirooih@gmail.com>
+
+ 12/4
+ ----
+aclocal.m4
+ - BASH_FUNC_FNMATCH_EQUIV_FALLBACK: a test of whether fnmatch(3)
+ understands bracket equivalence classes ([=c=]) for characters
+ that collate with equal weights but are not identical
+
+configure.ac,config.h.in
+ - call BASH_FUNC_FNMATCH_EQUIV_FALLBACK and define
+ FNMATCH_EQUIV_FALLBACK to 1 if it can be used for equivalence
+ classes
+
+ 12/5
+ ----
+execute_cmd.c
+ - eval_arith_for_expr,execute_arith_command,execute_cond_command: make
+ sure running_trap == 0 before we reset the_printed_command_except_trap
+ Report from Peng Yu <pengyu.ut@gmail.com>
+
+lib/glob/smatch.c
+ - _fnmatch_fallback_wc: new function, takes two wide characters c1 and
+ c2, converts them to a pattern ([[=c2=]]) and a string (c1) for
+ fnmatch to determine whether or not they are members of the same
+ equivalence class
+ - collequiv_wc: call _fnmatch_fallback_wc if rangecmp_wc returns
+ non-zero if FNMATCH_EQUIV_FALLBACK is defined, so we know that
+ fnmatch understands equivalence classes. Another Posix test suite
+ issue from Martin Rehak <martin.rehak@oracle.com>
+
+ 12/6
+ ----
+redir.c
+ - add missing cases to switch statements to shut up gcc
+
+ 12/7
+ ----
+builtins/set.def
+ - find_minus_o_option: new helper function, returns index into
+ o_options given option name
+ - minus_o_option_value,set_minus_o_option: use find_minus_o_option
+
+general.c
+ - new table of variables (currently all shopt options) that are
+ modified by going into and out of posix mode; num_posix_options()
+ returns the number of variables
+ - get_posix_options: fill in a bitmap passed as an argument (or return
+ a new one) of values of posix-mode-modified variables in the table
+ - set_posix_options: set values of posix-mode-modified variables from
+ the table using the passed bitmap for values
+
+builtins/set.def
+ - get_current_options: make the bitmap large enough to hold the options
+ in the set table and the table of posix-mode-modified variables; call
+ get_posix_options to fill in those values after the values from the
+ o_options table
+ - set_current_options: call set_posix_options to reset the values of
+ the posix-mode-modified variables at the end of the bitmap, after
+ the o_options values. Fixes issue reported by PJ Eby
+ <pje@telecommunity.com>
+
+ 12/9
+ ----
+parse.y
+ - select_command: add two additional productions to support select
+ commands without a word_list following the `in'. Fixes omission
+ reported by Martijn Dekker <martijn@inlv.org>
+
+ 12/11
+ -----
+variables.c
+ - assign_in_env: don't allow namerefs in temporary environment
+ assignments to create variables with invalid names for export. Fixes
+ bug reported by Grisha Levit <grishalevit@gmail.com>
+
+ 12/14
+ -----
+parse.y
+ - don't change last_command_exit_value in the 'error yacc_EOF' production
+ if the parser sets it to something non-zero; just make sure it
+ indicates an error. Fixes problem with unexpected EOF in eval
+ reported by Martijn Dekker <martijn@inlv.org>
+
+ 12/17
+ -----
+expr.c
+ - exp2 -> expmuldiv
+
+lib/sh/smatch.c
+ - fnmatch: add extern declaration if FNMATCH_EQUIV_FALLBACK is being
+ used
+
+hashlib.c
+ - hash_string: add FNV offset as initial value for hash instead of
+ starting at 0, changes hash for associative arrays and requires a
+ bunch of changes to the "right" test suite files
+
+variables.c
+ - rseed: make it explicitly 32 bits; that's all we're interested in
+ anyway
+
+ 12/18
+ -----
+
+variables.c
+ - brand: add some comments, make the constants explicitly 32 bits;
+ use signed 32-bit values for the intermediate calculations. The
+ algorithm is still minstd
+
+[bash-5.0-rc1 frozen]
+
+ 12/21
+ -----
+doc/bash.1
+ - }1: remove macro definition; it's outlived its bugfix purpose.
+ Report from Derek Schrock <dereks@lifeofadishwasher.com>
+
+ 12/22
+ -----
+subst.c
+ - expand_arith_string: set W_NOTILDE in the flags so we don't perform
+ tilde expansion at all, even for the previously-special-case array
+ subscript expansion. Report from Bize Ma <binaryzebra@gmail.com>
+ - expand_word_internal: take out Q_ARRAYSUB check for tilde expansion
+
+ 12/26
+ -----
+builtins/evalstring.c
+ - parse_and_execute: if the eval builtin gets a parser error while
+ parsing a string in posix mode, don't exit the shell if the eval
+ was run by the command builtin. report from Martijn Dekker
+ <martijn@inlv.org>
+
+examples/loadables/{basename,dirname}.c
+ - dirname_builtin: skip over any `--' ending the options. Report from
+ Peng Yu <pengyu.ut@gmail.com>
+
+ 12/27
+ -----
+examples/loadables/mkdir.c
+ - make_path: add argument noting whether or not the user specified -m;
+ only attempt the chmod on an existing directory if the user did so
+ - make_path: when creating intermediate directories, perform the
+ mkdir (path, 0) and chmod separately as the posix text recommends
+
+ 12/28
+ -----
+parser.h
+ - PST_COMMENT: new state, set when the shell is reading characters
+ until newline as part of comment processing
+
+parse.y
+ - shell_getc: don't return a space at the end of a token if the parser
+ is consuming a comment. Fixes bug reported by Harald van Dijk
+ <harald@gigawatt.nl>
+
+ 12/31
+ -----
+lib/glob/glob_loop.c
+ - INTERNAL_GLOB_PATTERN_P: revert change from 4/27 that makes this
+ function return non-zero for a backslash in the string. Based on a
+ report from Tom Ryder <tom@sanctum.geek.nz>
+
+[bash-5.0 frozen]