diff options
author | Mary Ruthven <mruthven@chromium.org> | 2021-04-01 07:48:53 +1100 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-04-26 16:46:53 +0000 |
commit | 3a6ddb7ccd7c0d6ead740b4ab91853a791ea96bf (patch) | |
tree | c1059612c36c54958720c79bc6d4477d6b0feada | |
parent | d624d0999f651c0b749134ad97143b0cab45b255 (diff) | |
download | chrome-ec-3a6ddb7ccd7c0d6ead740b4ab91853a791ea96bf.tar.gz |
make rddkeepalive read-only when console is locked
Print the Rdd state when the console is locked. Don't allow setting it.
BUG=none
TEST=manual
> ccd lock
CCD locked.
> rddkeepalive
Rdd: connected
> rddkeepalive enable
Parameter 1 invalid
Usage: rddkeepalive [BOOLEAN]
> ccd testlab open
> ccd
State: Opened
...
> rddkeepalive enable
Forcing Rdd detect keepalive
>
Change-Id: Ie309e42e7ef295bb0d7e7208d4bd6c17ce799e98
Signed-off-by: Mary Ruthven <mruthven@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2799440
Reviewed-by: Namyoon Woo <namyoon@chromium.org>
(cherry picked from commit 27c3bf4011f017b296fe3c49ae35ff6d9960b962)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2837409
(cherry picked from commit b038e3235ba4d2cafb6f42282f4996bb399ea0c4)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2851341
-rw-r--r-- | chip/g/rdd.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/chip/g/rdd.c b/chip/g/rdd.c index c58dce4806..4e17cab29a 100644 --- a/chip/g/rdd.c +++ b/chip/g/rdd.c @@ -214,7 +214,7 @@ static int command_rdd_keepalive(int argc, char **argv) return EC_SUCCESS; } - if (!parse_bool(argv[1], &force_detected)) + if (console_is_restricted() || !parse_bool(argv[1], &force_detected)) return EC_ERROR_PARAM1; if (force_detected) { @@ -228,6 +228,6 @@ static int command_rdd_keepalive(int argc, char **argv) return EC_SUCCESS; } -DECLARE_CONSOLE_COMMAND(rddkeepalive, command_rdd_keepalive, - "[BOOLEAN]", - "Get Rdd state or force keepalive"); +DECLARE_SAFE_CONSOLE_COMMAND(rddkeepalive, command_rdd_keepalive, + "[BOOLEAN]", + "Get Rdd state or force keepalive"); |