Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix command substitutions to parse contents as they are read in. | Peter Stephenson | 2015-01-08 | 5 | -193/+359 |
| | | | | | | | Do this by refactoring misnamed lexsave()/lexrestore() to allow continuity of history and input. Add test. | ||||
* | 34154/34155: reorder bin_print() to avoid leaking the output descriptor when ↵ | Barton E. Schaefer | 2015-01-07 | 1 | -29/+34 |
| | | | | incorrect/incompatible options were passed | ||||
* | 34144: allocate origline by ztrdup(), not by dupstring() | Jun-ichi Takimoto | 2015-01-07 | 2 | -3/+6 |
| | | | | | If origline is allocated in heap, it will have been freed when menuselect() is called directly as a widget. | ||||
* | 34122: module: allow NULL third argument as intended | Bart Schaefer | 2015-01-06 | 1 | -1/+1 |
| | | | | Found by Coverity. | ||||
* | 34138: wcs_nicechar: only deref widthp if it was given | Mikael Magnusson | 2015-01-06 | 1 | -1/+1 |
| | |||||
* | 34120: compctl, jobs: Check contents instead of array | Mikael Magnusson | 2015-01-06 | 2 | -2/+2 |
| | | | | | | text is an array in the struct, and can never be null. Found by Coverity (Issue 1255780). | ||||
* | 34108: Don't leak ifs stuff | Mikael Magnusson | 2015-01-06 | 1 | -2/+2 |
| | | | | Found by Coverity (Issue 1255785). | ||||
* | 34107: getsubsargs: free ptr1 before returning | Mikael Magnusson | 2015-01-06 | 1 | -0/+1 |
| | | | | Found by Coverity (Issue 439073). | ||||
* | 34134: anon funcs: don't leak shf and related data | Mikael Magnusson | 2015-01-06 | 1 | -0/+6 |
| | | | | Found by Coverity (Issue 439076). | ||||
* | 34112: typeset: fix leak of oldval | Mikael Magnusson | 2015-01-06 | 1 | -0/+2 |
| | | | | Found by Coverity (Issue 1255803). | ||||
* | 34106: hist: use zhtricat instead of tricat | Mikael Magnusson | 2015-01-06 | 1 | -1/+1 |
| | | | | Found by Coverity (Issue 1255769). | ||||
* | 34113: whence: use dupstring to not leak memory | Mikael Magnusson | 2015-01-06 | 1 | -1/+1 |
| | | | | | All other assignments to buf use the heap, and it's never freed. Found by Coverity (Issue 1255786). | ||||
* | 34119: complist: Fix leak of string in clnicezputs | Mikael Magnusson | 2015-01-06 | 1 | -0/+2 |
| | | | | Found by Coverity (Issue 1255808). | ||||
* | 34105: subst: remove dead code | Mikael Magnusson | 2015-01-06 | 1 | -1/+1 |
| | | | | Found by Coverity (Issue 1255810). | ||||
* | 34121: compresult: Remove unneeded NULL check | Mikael Magnusson | 2015-01-06 | 1 | -1/+1 |
| | | | | | The variable is set to if NULL at the start of the function, and derefed on the previous line. Found by Coverity (Issue 1255843). | ||||
* | 34104: compctl: Remove pointless check | Mikael Magnusson | 2015-01-06 | 1 | -1/+1 |
| | | | | | cc has already been derefed a bunch of times leading up to here. Found by Coverity (Issue 1255841). | ||||
* | 34115: compcore: Fix size argument to zfree | Mikael Magnusson | 2015-01-06 | 1 | -1/+1 |
| | | | | | Found by Coverity (Issue 1255852), has no impact unless using --enable-zsh-mem, and even then it is minimal. | ||||
* | 34117: zle: size_t is unsigned, use int instead | Mikael Magnusson | 2015-01-06 | 1 | -1/+1 |
| | | | | | | The function wctomb returns an int according to my manpage, and we furthermore check if it is negative, and then return it, and the function signature is int, so declaring it as an int seems to make more sense. | ||||
* | 34116: computil: Check for NULL before passing to strlen | Mikael Magnusson | 2015-01-06 | 1 | -1/+2 |
| | | | | | The rest of this function appears to be very careful about checking these, then forgets in this one spot. Found by Coverity (Issue 1255805). | ||||
* | 34118: Don't crash when writing out history if HOST is unset | Mikael Magnusson | 2015-01-06 | 1 | -1/+2 |
| | | | | Found by Coverity (Issue 1255793). | ||||
* | 34114: emulate: Handle aborting from mixed -L/-c correctly | Mikael Magnusson | 2015-01-06 | 1 | -1/+2 |
| | | | | Somehow Coverity found this (Issue 1255797, Failure to restore non-local value). | ||||
* | 34103: fix ancient double-quote handling thinko in subst_parse_str() | Barton E. Schaefer | 2015-01-06 | 1 | -2/+4 |
| | | | | This doesn't seem to have mattered, but must in some obscure cases | ||||
* | 34093: "whence" should always return nonzero when it finds that nothing ↵ | Barton E. Schaefer | 2015-01-06 | 1 | -15/+17 |
| | | | | matches its arguments | ||||
* | 34092: fix miscount of symlink resolution for "..". | Peter Stephenson | 2015-01-04 | 1 | -0/+2 |
| | | | | This caused problems with expanding a path with ".." in "whence -S". | ||||
* | 34091: typo with "whence -s" expansions | Peter Stephenson | 2015-01-04 | 1 | -2/+2 |
| | |||||
* | users/19671: remove confusion with whence -a. | Peter Stephenson | 2015-01-02 | 1 | -1/+1 |
| | | | | If the argument is a full path don't try to search the path for it. | ||||
* | users/19667: whence -S shows intermediate steps in symlink expansion | Peter Stephenson | 2015-01-02 | 2 | -13/+66 |
| | |||||
* | 34070: fix starting position for memset() from 34005. | Daniel Shahaf | 2014-12-28 | 1 | -1/+1 |
| | |||||
* | 34064: assignment before command replaces array with export even when KSH_ARRAYS | Takeshi Banse | 2014-12-27 | 1 | -0/+2 |
| | |||||
* | 34065: following an "if" condition, do not test lastval for ERR_EXIT until a ↵ | Barton E. Schaefer | 2014-12-27 | 2 | -5/+15 |
| | | | | | | new command is run Includes unposted regression tests. | ||||
* | 34015: disallow strange environment variable names. | Peter Stephenson | 2014-12-19 | 1 | -16/+23 |
| | | | | | These are ones with the top bit set in any character. Don't import them, and don't export them. | ||||
* | 34005: region_highlights memory fix | Peter Stephenson | 2014-12-19 | 1 | -2/+8 |
| | | | | Zero uninitialised part of memory when reallocing | ||||
* | 34002: zshcalloc() in init_keymaps() | Barton E. Schaefer | 2014-12-18 | 1 | -1/+1 |
| | |||||
* | 33992: do not attempt attachtty() for process group zero (which is possible ↵ | Barton E. Schaefer | 2014-12-18 | 1 | -2/+5 |
| | | | | in a linux pid namespace) | ||||
* | 34008: metafy the environment on arrival in the shell | Peter Stephenson | 2014-12-18 | 1 | -1/+1 |
| | |||||
* | 34006: unmetafy anything put into the environment | Peter Stephenson | 2014-12-18 | 1 | -1/+11 |
| | |||||
* | 33981: more care with region_highlights management | Peter Stephenson | 2014-12-18 | 2 | -29/+39 |
| | |||||
* | 33978: avoid infinite loop in interactive mode | Jun-ichi Takimoto | 2014-12-18 | 1 | -1/+3 |
| | | | | | Update command line when accept-and-hold is called in the interactive mode of menu select. | ||||
* | 33984: bin_dirs() should use zputs() to print metafied directory names | Jun Kuriyama | 2014-12-16 | 1 | -1/+1 |
| | |||||
* | 33982: minimal support for pid namespaces by recognizing that GETPGRP() may ↵ | Chirantan Ekbote | 2014-12-16 | 1 | -1/+1 |
| | | | | return 0 | ||||
* | 33976: fix overlapping strcpy() | Barton E. Schaefer | 2014-12-15 | 1 | -1/+1 |
| | |||||
* | 33956: document key binding changes and remove ^X binding | Oliver Kiddle | 2014-12-13 | 2 | -1/+2 |
| | |||||
* | 33846: additional default vi-mode key bindings | Oliver Kiddle | 2014-12-13 | 3 | -5/+12 |
| | |||||
* | 33950: ignore KEYTIMEOUT for vi operators | Oliver Kiddle | 2014-12-12 | 3 | -12/+19 |
| | |||||
* | 33876: etc.: Separate errors and keyboards interrupts | Peter Stephenson | 2014-12-11 | 28 | -124/+273 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Combination of 12 commits from interrupt_abort branch. Basic strategy is to introduce bits to errflag and to set and reset them separately. Remove interrupt status on return to main keymap. Turn off ERRFLAG_INT for always block. Restore bit thereafter: we probably need a new variable in order to allow user interrupts to be reset in the always block. Add TRY_BLOCK_INTERRUPT This works the same as TRY_BLOCK_ERROR, but for a SIGINT, too. Ensure propagation of SIGINT from exited job. If received by foreground job, shell uses ERRFLAG_INT, not ERRFLAG_ERROR, to set the new state. Reset errflag before precmd() Add always block in _main_completion to fix ZLS_COLORS Ensures we get the right state of $ZLS_COLORS at the end of _main_complete even if there's an interrupt. However, the "right state" is a bit messy as it depends on styles. | ||||
* | 33940: handle backslash-meta correctly in lexer | Jun-ichi Takimoto | 2014-12-10 | 1 | -1/+13 |
| | |||||
* | 33932: revise boundary check in unmeta() | Jun-ichi Takimoto | 2014-12-09 | 1 | -5/+3 |
| | |||||
* | 33924: allow vi line/characterwise mode to be forced | Oliver Kiddle | 2014-12-08 | 4 | -3/+26 |
| | |||||
* | 33894: boundary conditions in unmeta(), unmetafy() | Barton E. Schaefer | 2014-12-07 | 1 | -3/+5 |
| | | | | | | | Check that we aren't running off the end of the string when converting the next byte after a Meta byte. This is just defensive programming in case of bad metafied strings coming through from gettokstr(), some repairs there are likely still needed. | ||||
* | 33854: errors end recursion through scanner() | Barton E. Schaefer | 2014-12-07 | 1 | -1/+1 |
| | | | | Makes **/ more easily interruptible |