]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Correct assert added to dump program.
authorKirk McKusick <mckusick@FreeBSD.org>
Mon, 17 May 2021 23:33:59 +0000 (16:33 -0700)
committerKirk McKusick <mckusick@FreeBSD.org>
Mon, 17 May 2021 23:34:53 +0000 (16:34 -0700)
commitefe145a7453e4208f032816ce3f80e9fb6b0e4ee
treea51838a0c1317d29e7b460bd28e497502755d2f0
parentf5b5de1a3210234f3a6864c88a2d3e11ac2dbf04
Correct assert added to dump program.

The dump program was exiting with the message:

Assertion failed: (spcl.c_count + blks < TP_NINDIR), function appendextdata, file /usr/src/sbin/dump/traverse.c, line 759.

The problem arose when dumping external attributes.

This assertion was added in this commit with no review by someone
with expertise in the dump program:

    commit 2d518c6518cdb256ff6f2c463e6b115d89c104c3
    Author:     Warner Losh <imp@FreeBSD.org>
    AuthorDate: Mon Jun 11 19:32:36 2018 +0000
    Commit:     Warner Losh <imp@FreeBSD.org>
    CommitDate: Mon Jun 11 19:32:36 2018 +0000

Add asserts to prevent overflows of c_addr.

It is clearly wrong as the statement immediately above it in the
code which is deciding if the data will fit is:

if (spcl.c_count + blks > TP_NINDIR)
return (0);

As is pointed out in the bug report, the assert should be:

(spcl.c_count + blks <= TP_NINDIR)

This commit corrects the assert. I am sorry that it took so long to
be brought to my attention and get fixed.

Reported by:  Hampton Finger
PR:           244470
MFC after:    3 days
Sponsored by: Netflix
sbin/dump/traverse.c