mirror of
https://git.FreeBSD.org/src.git
synced 2024-12-05 09:14:03 +00:00
Back out previous delta to fix fsck on filesystems without an fstab entry,
where we want to take the disklabel filesystem type of "4.2BSD" and use fsck_4.2bsd on those filesystems. Add a comment about why the code is there, now that we know: * XXX This is a kludge to make automatic filesystem type guessing * from the disklabel work for "4.2BSD" filesystems. It does a * very limited subset of transliteration to a normalised form of * filesystem name, and we do not seem to enforce a filesystem * name character set.
This commit is contained in:
parent
913fc94d2b
commit
8235d79a86
Notes:
svn2git
2020-12-20 02:59:44 +00:00
svn path=/head/; revision=113994
@ -278,7 +278,7 @@ isok(struct fstab *fs)
|
|||||||
|
|
||||||
|
|
||||||
static int
|
static int
|
||||||
checkfs(const char *vfstype, const char *spec, const char *mntpt,
|
checkfs(const char *pvfstype, const char *spec, const char *mntpt,
|
||||||
char *auxopt, pid_t *pidp)
|
char *auxopt, pid_t *pidp)
|
||||||
{
|
{
|
||||||
/* List of directories containing fsck_xxx subcommands. */
|
/* List of directories containing fsck_xxx subcommands. */
|
||||||
@ -291,6 +291,7 @@ checkfs(const char *vfstype, const char *spec, const char *mntpt,
|
|||||||
pid_t pid;
|
pid_t pid;
|
||||||
int argc, i, status, maxargc;
|
int argc, i, status, maxargc;
|
||||||
char *optbuf, execname[MAXPATHLEN + 1], execbase[MAXPATHLEN];
|
char *optbuf, execname[MAXPATHLEN + 1], execbase[MAXPATHLEN];
|
||||||
|
char *vfstype = NULL;
|
||||||
const char *extra = NULL;
|
const char *extra = NULL;
|
||||||
|
|
||||||
#ifdef __GNUC__
|
#ifdef __GNUC__
|
||||||
@ -298,6 +299,24 @@ checkfs(const char *vfstype, const char *spec, const char *mntpt,
|
|||||||
(void) &optbuf;
|
(void) &optbuf;
|
||||||
(void) &vfstype;
|
(void) &vfstype;
|
||||||
#endif
|
#endif
|
||||||
|
/*
|
||||||
|
* We convert the vfstype to lowercase and any spaces to underscores
|
||||||
|
* to not confuse the issue
|
||||||
|
*
|
||||||
|
* XXX This is a kludge to make automatic filesystem type guessing
|
||||||
|
* from the disklabel work for "4.2BSD" filesystems. It does a
|
||||||
|
* very limited subset of transliteration to a normalised form of
|
||||||
|
* filesystem name, and we do not seem to enforce a filesystem
|
||||||
|
* name character set.
|
||||||
|
*/
|
||||||
|
vfstype = strdup(pvfstype);
|
||||||
|
if (vfstype == NULL)
|
||||||
|
perror("strdup(pvfstype)");
|
||||||
|
for (i = 0; i < strlen(vfstype); i++) {
|
||||||
|
vfstype[i] = tolower(vfstype[i]);
|
||||||
|
if (vfstype[i] == ' ')
|
||||||
|
vfstype[i] = '_';
|
||||||
|
}
|
||||||
|
|
||||||
extra = getoptions(vfstype);
|
extra = getoptions(vfstype);
|
||||||
optbuf = NULL;
|
optbuf = NULL;
|
||||||
@ -334,6 +353,7 @@ checkfs(const char *vfstype, const char *spec, const char *mntpt,
|
|||||||
warn("vfork");
|
warn("vfork");
|
||||||
if (optbuf)
|
if (optbuf)
|
||||||
free(optbuf);
|
free(optbuf);
|
||||||
|
free(vfstype);
|
||||||
return (1);
|
return (1);
|
||||||
|
|
||||||
case 0: /* Child. */
|
case 0: /* Child. */
|
||||||
@ -367,6 +387,8 @@ checkfs(const char *vfstype, const char *spec, const char *mntpt,
|
|||||||
if (optbuf)
|
if (optbuf)
|
||||||
free(optbuf);
|
free(optbuf);
|
||||||
|
|
||||||
|
free(vfstype);
|
||||||
|
|
||||||
if (pidp) {
|
if (pidp) {
|
||||||
*pidp = pid;
|
*pidp = pid;
|
||||||
return 0;
|
return 0;
|
||||||
|
Loading…
Reference in New Issue
Block a user