cli_hush.c: remove broken sanity check
authorRasmus Villemoes <rasmus.villemoes@prevas.dk>
Wed, 7 Oct 2020 07:36:03 +0000 (09:36 +0200)
committerTom Rini <trini@konsulko.com>
Thu, 22 Oct 2020 13:54:53 +0000 (09:54 -0400)
commit8d5d97cb28bf2ac1e0d7e4b24620c56f3fe45e6f
tree780894134c15c46440d4d9bc185aaaa053f06b6f
parent6ebe6b387f63a17cecc91fa05284b577fe0d45f1
cli_hush.c: remove broken sanity check

This code is intended do prevent one from setting a shell variable abc
by doing

  abc=123

if an environment variable named abc already exists. However, the
check is broken, since the env_get is done before the split on =, so
we look up whether an environment variable "abc=123" exists, which is
obviously never the case.

One could move the code to below the split on =, but instead, just
remove it, saving a little .text: The check has never worked as
intended (it goes all the way back to the initial git commit), and it
would anyway not guard against one first setting the shell variable,
then doing 'env set abc xyz'.

Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
common/cli_hush.c