]> git.ipfire.org Git - thirdparty/shadow.git/commit
Fix covscan BUFFER_SIZE 362/head
authorIker Pedrosa <ipedrosa@redhat.com>
Wed, 16 Jun 2021 07:50:53 +0000 (09:50 +0200)
committerIker Pedrosa <ipedrosa@redhat.com>
Wed, 16 Jun 2021 07:50:53 +0000 (09:50 +0200)
commit5d0d7841971cc53d9a9d1aefe12f00204115bf6a
tree6582811149869d0c6624b4e303d43f34d47e8d48
parent456bdb78f886987f64045aa032277423f8888bd6
Fix covscan BUFFER_SIZE

Error: BUFFER_SIZE (CWE-170): [#def6]
shadow-4.8.1/libmisc/failure.c:101: buffer_size_warning: Calling "strncpy" with a maximum size argument of 12 bytes on destination array "fl->fail_line" of size 12 bytes might leave the destination string unterminated.
   99|    }
  100|
  101|->  strncpy (fl->fail_line, tty, sizeof fl->fail_line);
  102|    (void) time (&fl->fail_time);
  103|

Error: BUFFER_SIZE (CWE-170): [#def9]
shadow-4.8.1/libmisc/log.c:103: buffer_size_warning: Calling "strncpy" with a maximum size argument of 32 bytes on destination array "newlog.ll_line" of size 32 bytes might leave the destination string unterminated.
  101|    (void) time (&ll_time);
  102|    newlog.ll_time = ll_time;
  103|->  strncpy (newlog.ll_line, line, sizeof newlog.ll_line);
  104|   #if HAVE_LL_HOST
  105|    strncpy (newlog.ll_host, host, sizeof newlog.ll_host);

Error: BUFFER_SIZE (CWE-170): [#def10]
shadow-4.8.1/libmisc/log.c:105: buffer_size_warning: Calling "strncpy" with a maximum size argument of 256 bytes on destination array "newlog.ll_host" of size 256 bytes might leave the destination string unterminated.
  103|    strncpy (newlog.ll_line, line, sizeof newlog.ll_line);
  104|   #if HAVE_LL_HOST
  105|->  strncpy (newlog.ll_host, host, sizeof newlog.ll_host);
  106|   #endif
  107|    if (   (lseek (fd, offset, SEEK_SET) != offset)

Error: BUFFER_SIZE (CWE-170): [#def13]
shadow-4.8.1/libmisc/utmp.c:260: buffer_size_warning: Calling "strncpy" with a maximum size argument of 32 bytes on destination array "utent->ut_line" of size 32 bytes might leave the destination string unterminated.
  258|   #endif /* HAVE_STRUCT_UTMP_UT_TYPE */
  259|    utent->ut_pid = getpid ();
  260|->  strncpy (utent->ut_line, line,      sizeof (utent->ut_line));
  261|   #ifdef HAVE_STRUCT_UTMP_UT_ID
  262|    if (NULL != ut) {

Error: BUFFER_SIZE (CWE-170): [#def14]
shadow-4.8.1/libmisc/utmp.c:266: buffer_size_warning: Calling "strncpy" with a maximum size argument of 4 bytes on destination array "utent->ut_id" of size 4 bytes might leave the destination string unterminated.
  264|    } else {
  265|    /* XXX - assumes /dev/tty?? */
  266|->  strncpy (utent->ut_id, line + 3, sizeof (utent->ut_id));
  267|    }
  268|   #endif /* HAVE_STRUCT_UTMP_UT_ID */

Error: BUFFER_SIZE (CWE-170): [#def15]
shadow-4.8.1/libmisc/utmp.c:273: buffer_size_warning: Calling "strncpy" with a maximum size argument of 32 bytes on destination array "utent->ut_user" of size 32 bytes might leave the destination string unterminated.
  271|   #endif /* HAVE_STRUCT_UTMP_UT_NAME */
  272|   #ifdef HAVE_STRUCT_UTMP_UT_USER
  273|->  strncpy (utent->ut_user, name,      sizeof (utent->ut_user));
  274|   #endif /* HAVE_STRUCT_UTMP_UT_USER */
  275|    if (NULL != hostname) {

Error: BUFFER_SIZE (CWE-170): [#def16]
shadow-4.8.1/libmisc/utmp.c:278: buffer_size_warning: Calling "strncpy" with a maximum size argument of 256 bytes on destination array "utent->ut_host" of size 256 bytes might leave the destination string unterminated.
  276|    struct addrinfo *info = NULL;
  277|   #ifdef HAVE_STRUCT_UTMP_UT_HOST
  278|->  strncpy (utent->ut_host, hostname, sizeof (utent->ut_host));
  279|   #endif /* HAVE_STRUCT_UTMP_UT_HOST */
  280|   #ifdef HAVE_STRUCT_UTMP_UT_SYSLEN

Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com>
libmisc/failure.c
libmisc/log.c
libmisc/utmp.c