From a173916590ffbc512de8d0e99358e91e096f1877 Mon Sep 17 00:00:00 2001 From: Gleb Smirnoff Date: Tue, 15 Oct 2013 13:32:01 +0000 Subject: [PATCH] Make getutxent(3) more robust against bad utx.log files. Whenever we read zeroes, don't stop processing the file, but read until its end or valid data. In collaboration with: ed --- lib/libc/gen/getutxent.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/lib/libc/gen/getutxent.c b/lib/libc/gen/getutxent.c index 20e8859f6cd7..1679aa43d097 100644 --- a/lib/libc/gen/getutxent.c +++ b/lib/libc/gen/getutxent.c @@ -122,9 +122,20 @@ getfutxent(struct futx *fu) if (udb == UTXDB_LOG) { uint16_t len; +retry: if (fread(&len, sizeof(len), 1, uf) != 1) return (-1); len = be16toh(len); + if (len == 0) { + /* + * XXX: Though zero-size records are valid in theory, + * they can never occur in practice. Zero-size records + * indicate file corruption. Seek one byte forward, to + * see if we can find a record there. + */ + ungetc('\0', uf); + goto retry; + } if (len > sizeof *fu) { /* Forward compatibility. */ if (fread(fu, sizeof(*fu), 1, uf) != 1)