From a8a00be442370ce491f9cc22a3fa0b4853feecb5 Mon Sep 17 00:00:00 2001 From: Daniel Shahaf Date: Wed, 13 Jan 2016 00:57:55 +0000 Subject: 37591: 'alias -L': skip with a warning aliases with '=' in their LHS --- ChangeLog | 3 +++ Src/hashtable.c | 9 +++++++++ Test/A02alias.ztst | 8 ++++++++ 3 files changed, 20 insertions(+) diff --git a/ChangeLog b/ChangeLog index 7309cb3f5..731e323ab 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ 2016-01-13 Daniel Shahaf + * 37591: Src/hashtable.c, Test/A02alias.ztst: 37591: 'alias -L': + skip with a warning aliases with '=' in their LHS + * 37550: Completion/BSD/Command/_cu: _cu: Support Linux line-device names; fail gracefully on OSes matching no known line-device name pattern. diff --git a/Src/hashtable.c b/Src/hashtable.c index 2d1ff87cb..0664c3694 100644 --- a/Src/hashtable.c +++ b/Src/hashtable.c @@ -1276,6 +1276,15 @@ printaliasnode(HashNode hn, int printflags) } if (printflags & PRINT_LIST) { + /* Fast fail on unrepresentable values. */ + if (strchr(a->node.nam, '=')) { + zwarn("invalid alias '%s' encountered while printing aliases", + a->node.nam); + /* ### TODO: Return an error status to the C caller */ + return; + } + + /* Normal path. */ printf("alias "); if (a->node.flags & ALIAS_SUFFIX) printf("-s "); diff --git a/Test/A02alias.ztst b/Test/A02alias.ztst index 389617898..49e47567c 100644 --- a/Test/A02alias.ztst +++ b/Test/A02alias.ztst @@ -96,3 +96,11 @@ 0:unalias -as >foo is a suffix alias for print >foo: suffix alias + + aliases[x=y]=z + alias -L | grep x=y + echo $pipestatus[1] +0:printing invalid aliases warns +>0 +?(eval):2: invalid alias 'x=y' encountered while printing aliases +# Currently, 'alias -L' returns 0 in this case. Perhaps it should return 1. -- cgit v1.2.3-70-g09d2