diff options
| author | Zelalem <zelalem.aweke@arm.com> | 2020-02-05 14:12:39 -0600 |
|---|---|---|
| committer | Zelalem <zelalem.aweke@arm.com> | 2020-02-05 14:53:02 -0600 |
| commit | 466bb285c6985027c75a230e39f2ae246fd07971 (patch) | |
| tree | 593c3f6465c7fdd981c452b5034ab8edf75b7205 /drivers/console/multi_console.c | |
| parent | 235c8174ffe55bfc1108c02e0b41ca9916428f21 (diff) | |
| download | platform_external_arm-trusted-firmware-466bb285c6985027c75a230e39f2ae246fd07971.tar.gz platform_external_arm-trusted-firmware-466bb285c6985027c75a230e39f2ae246fd07971.tar.bz2 platform_external_arm-trusted-firmware-466bb285c6985027c75a230e39f2ae246fd07971.zip | |
coverity: Fix MISRA null pointer violations
Fix code that violates the MISRA rule:
MISRA C-2012 Rule 11.9: Literal "0" shall not be used as
null pointer constant.
The fix explicitly checks whether a pointer is NULL.
Change-Id: Ibc318dc0f464982be9a34783f24ccd1d44800551
Signed-off-by: Zelalem <zelalem.aweke@arm.com>
Diffstat (limited to 'drivers/console/multi_console.c')
| -rw-r--r-- | drivers/console/multi_console.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/console/multi_console.c b/drivers/console/multi_console.c index 215f49517..0665f202f 100644 --- a/drivers/console/multi_console.c +++ b/drivers/console/multi_console.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2018-2019, ARM Limited and Contributors. All rights reserved. + * Copyright (c) 2018-2020, ARM Limited and Contributors. All rights reserved. * * SPDX-License-Identifier: BSD-3-Clause */ @@ -90,7 +90,7 @@ int console_putc(int c) console_t *console; for (console = console_list; console != NULL; console = console->next) - if ((console->flags & console_state) && console->putc) { + if ((console->flags & console_state) && (console->putc != NULL)) { int ret = do_putc(c, console); if ((err == ERROR_NO_VALID_CONSOLE) || (ret < err)) err = ret; @@ -107,7 +107,7 @@ int console_getc(void) do { /* Keep polling while at least one console works correctly. */ for (console = console_list; console != NULL; console = console->next) - if ((console->flags & console_state) && console->getc) { + if ((console->flags & console_state) && (console->getc != NULL)) { int ret = console->getc(console); if (ret >= 0) return ret; @@ -125,7 +125,7 @@ int console_flush(void) console_t *console; for (console = console_list; console != NULL; console = console->next) - if ((console->flags & console_state) && console->flush) { + if ((console->flags & console_state) && (console->flush != NULL)) { int ret = console->flush(console); if ((err == ERROR_NO_VALID_CONSOLE) || (ret < err)) err = ret; |