diff options
author | Ryan Long <ryan.long@oarcorp.com> | 2021-02-19 17:06:38 -0500 |
---|---|---|
committer | Joel Sherrill <joel@rtems.org> | 2021-02-26 16:56:07 -0600 |
commit | 1b322a6b83990b9c8fc5ba05d78edf8bba533ca1 (patch) | |
tree | c58476156c2cfff892b8d13524369f0d0081aa9d | |
parent | build: Install <rtems/score/hash.h> (diff) | |
download | rtems-1b322a6b83990b9c8fc5ba05d78edf8bba533ca1.tar.bz2 |
printertask.c: Fix Unchecked return value (CID #1399710)
CID 1399710: Unchecked return value in printer_task().
CID 1472764: Unchecked return value in rtems_print_printer_task().
Closes #4258
-rw-r--r-- | cpukit/libcsupport/src/printertask.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/cpukit/libcsupport/src/printertask.c b/cpukit/libcsupport/src/printertask.c index bae623bff3..31d569f321 100644 --- a/cpukit/libcsupport/src/printertask.c +++ b/cpukit/libcsupport/src/printertask.c @@ -117,6 +117,8 @@ static void printer_task( rtems_task_argument arg ) { rtems_printer_task_context *ctx; int fd; + int err; + rtems_status_code sc; ctx = (rtems_printer_task_context *) arg; fd = ctx->fd; @@ -141,8 +143,10 @@ static void printer_task( rtems_task_argument arg ) printer_task_append_buffer( ctx, &ctx->free_buffers, buffer ); break; case ACTION_DRAIN: - fsync(fd); - rtems_event_transient_send( buffer->action_data.task ); + err = fsync(fd); + _Assert_Unused_variable_equals(err, 0); + sc = rtems_event_transient_send( buffer->action_data.task ); + _Assert_Unused_variable_equals(sc, RTEMS_SUCCESSFUL); break; } } @@ -185,7 +189,8 @@ int rtems_print_printer_task( printer->context = ctx; printer->printer = printer_task_printer; - rtems_task_start( ctx->task, printer_task, (rtems_task_argument) ctx ); + sc = rtems_task_start( ctx->task, printer_task, (rtems_task_argument) ctx ); + _Assert_Unused_variable_equals(sc, RTEMS_SUCCESSFUL); return 0; } |