From 761d063e808061f78a2bd10e65cc88ff442124c6 Mon Sep 17 00:00:00 2001 From: phk Date: Wed, 18 Dec 2002 19:57:27 +0000 Subject: Fix two blunders in the mapping functions which can lead to corrupt data, for request sizes larger than the sectorsize or for multi-key setups. See warning mailed to current@ for details of recovery. Found by: Marcus Reid --- sbin/gbde/gbde.c | 22 +++++++++++++++++++++- 1 file changed, 21 insertions(+), 1 deletion(-) (limited to 'sbin/gbde') diff --git a/sbin/gbde/gbde.c b/sbin/gbde/gbde.c index e3e278e..041d206 100644 --- a/sbin/gbde/gbde.c +++ b/sbin/gbde/gbde.c @@ -418,7 +418,16 @@ cmd_write(struct g_bde_key *gl, struct g_bde_softc *sc, int dfd , int key, const if (i != (int)gl->sectorsize) err(1, "write"); printf("Wrote key %d at %jd\n", key, (intmax_t)offset); - +#if 0 + printf("s0 = %jd\n", (intmax_t)gl->sector0); + printf("sN = %jd\n", (intmax_t)gl->sectorN); + printf("l[0] = %jd\n", (intmax_t)gl->lsector[0]); + printf("l[1] = %jd\n", (intmax_t)gl->lsector[1]); + printf("l[2] = %jd\n", (intmax_t)gl->lsector[2]); + printf("l[3] = %jd\n", (intmax_t)gl->lsector[3]); + printf("k = %jd\n", (intmax_t)gl->keyoffset); + printf("ss = %jd\n", (intmax_t)gl->sectorsize); +#endif } static void @@ -436,6 +445,16 @@ cmd_destroy(struct g_bde_key *gl, int nkey) gl->lsector[i] = ~0; } +static int +sorthelp(const void *a, const void *b) +{ + const off_t *oa, *ob; + + oa = a; + ob = b; + return (*oa - *ob); +} + static void cmd_init(struct g_bde_key *gl, int dfd, const char *f_opt, int i_opt, const char *l_opt) { @@ -597,6 +616,7 @@ cmd_init(struct g_bde_key *gl, int dfd, const char *f_opt, int i_opt, const char while (o < gl->sectorN); gl->lsector[u] = o; } + qsort(gl->lsector, G_BDE_MAXKEYS, sizeof gl->lsector[0], sorthelp); /* Flush sector zero if we use it for lockfile data */ if (gl->flags & 1) { -- cgit v1.1