d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

print_config:

Browse Source 
Change config format slightly to save plex preferences correctly.

vinum_scandisk: reinitialise volatile pointer after function call.
This is the "deafc0de" bug.

Approved by: re (scottl)
This commit is contained in:
Greg Lehey 2003-05-19 02:21:31 +00:00
parent e92686d065
commit 4555a3de62
1 changed files with 14 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
sys/dev/vinum/vinumio.c
View File

@ -33,7 +33,7 @@
* otherwise) arising in any way out of the use of this software, even if
* advised of the possibility of such damage.
*
* $Id: vinumio.c,v 1.37 2003/05/04 05:23:42 grog Exp grog $
* $Id: vinumio.c,v 1.38 2003/05/07 03:29:30 grog Exp grog $
* $FreeBSD$
*/
@ -153,7 +153,7 @@ init_drive(struct drive *drive, int verbose)
if (drive->lasterror) {
if (verbose)
log(LOG_ERR,
"vinum: Can't get partition information for %s: error %d\n",
"vinum: Can't get drive dimensions for %s: error %d\n",
drive->devicename,
drive->lasterror);
close_drive(drive);
@ -482,10 +482,9 @@ format_config(char *config, int len)
drivename = "*invalid*";
snprintf(s,
configend - s,
"sd name %s drive %s plex %s len %llus driveoffset %llus state %s",
"sd name %s drive %s len %llus driveoffset %llus state %s",
sd->name,
drivename,
vinum_conf.plex[sd->plexno].name,
(unsigned long long) sd->sectors,
(unsigned long long) sd->driveoffset,
sd_state(sd->state));
@ -494,7 +493,8 @@ format_config(char *config, int len)
if (sd->plexno >= 0)
snprintf(s,
configend - s,
" plexoffset %llds",
" plex %s plexoffset %llds",
vinum_conf.plex[sd->plexno].name,
(long long) sd->plexoffset);
else
snprintf(s, configend - s, " detached");
@ -940,6 +940,14 @@ vinum_scandisk(char *devicename)
*eptr = '\0'; /* and delimit */
if (setjmp(command_fail) == 0) { /* come back here on error and continue */
parse_status = parse_config(config_line, &keyword_set, 1); /* parse the config line */
/*
* parse_config recognizes referenced
* drives and builds a drive entry for
* them. This may expand the drive
* table, thus invalidating the pointer.
*/
drive = &DRIVE[drivelist[driveno]]; /* point to the drive */
if (parse_status < 0) { /* error in config */
/*
* This config should have been parsed
@ -960,7 +968,7 @@ vinum_scandisk(char *devicename)
cptr++; /* skip to next line */
}
}
drive->flags |= VF_CONFIGURED; /* read this drive's configuration */
drive->flags |= VF_CONFIGURED; /* this drive's configuration is complete */
}
Free(config_text);