From 72e4efb46ee6a395f27ff54a64d3daaf24d85187 Mon Sep 17 00:00:00 2001 From: armink Date: Sat, 26 Nov 2016 16:45:54 +0800 Subject: [PATCH] [finsh]Improve the set password. --- components/finsh/shell.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/components/finsh/shell.c b/components/finsh/shell.c index 83709560b1..7e8a84ab27 100644 --- a/components/finsh/shell.c +++ b/components/finsh/shell.c @@ -33,6 +33,7 @@ * 2010-04-01 Bernard add prompt output when start and remove the empty history * 2011-02-23 Bernard fix variable section end issue of finsh shell * initialization when use GNU GCC compiler. + * 2016-11-26 armink add password authentication */ #include @@ -180,12 +181,14 @@ rt_uint32_t finsh_get_echo() * * @param password new password * - * @return result, RT_EOK on OK, -RT_ERROR on the new password length is less than FINSH_PASSWORD_MIN + * @return result, RT_EOK on OK, -RT_ERROR on the new password length is less than + * FINSH_PASSWORD_MIN or greater than FINSH_PASSWORD_MAX */ rt_err_t finsh_set_password(const char *password) { rt_ubase_t level; + rt_size_t pw_len = rt_strlen(password); - if (rt_strlen(password) < FINSH_PASSWORD_MIN) + if (pw_len < FINSH_PASSWORD_MIN || pw_len > FINSH_PASSWORD_MAX) return -RT_ERROR; level = rt_hw_interrupt_disable(); @@ -407,7 +410,7 @@ void finsh_thread_entry(void *parameter) #ifdef FINSH_USING_AUTH /* set the default password when the password isn't setting */ if (rt_strlen(finsh_get_password()) == 0) - finsh_set_password(FINSH_DEFAULT_PASSWORD); + RT_ASSERT(finsh_set_password(FINSH_DEFAULT_PASSWORD) == RT_EOK); /* waiting authenticate success */ finsh_wait_auth(); #endif