mirror/freebsd
1
0
Fork 0
mirror of https://git.FreeBSD.org/src.git synced 2025-01-22 15:47:37 +00:00
Code Issues Releases Activity

geli attach: Fix exit codes and errors not being printed after r335673

Browse Source 
Now that multiple providers can be attached at once, exit codes and
error messages must be handled correctly if there are failures in on
any of the providers.

Reported by:	asomers (Kyua test failures via continuous integration)
Reviewed by:	asomers
Approved by:	allanjude
Differential Revision:	https://reviews.freebsd.org/D16386
This commit is contained in:
Ben Woods 2018-07-22 13:40:52 +00:00
parent cca6b506e0
commit 4b8e4d53fa
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=336602
1 changed files with 20 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
lib/geom/eli/geom_eli.c
View File

@ -941,18 +941,26 @@ eli_attach(struct gctl_req *req)
prov = gctl_get_ascii(req, "arg%d", i);
gctl_ro_param(r, "arg0", -1, prov);
if (eli_metadata_read(r, prov, &md) == -1)
return;
if (eli_metadata_read(r, prov, &md) == -1) {
/*
* Error reading metadata - details added to geom
* request by eli_metadata_read().
*/
goto out;
}
mediasize = g_get_mediasize(prov);
if (md.md_provsize != (uint64_t)mediasize) {
gctl_error(r, "Provider size mismatch.");
return;
goto out;
}
if (eli_genkey(r, &md, key, false) == NULL) {
explicit_bzero(key, sizeof(key));
return;
/*
* Error generating key - details added to geom request
* by eli_genkey().
*/
goto out;
}
gctl_ro_param(r, "key", sizeof(key), key);
@ -962,19 +970,20 @@ eli_attach(struct gctl_req *req)
printf("Attached to %s.\n", prov);
}
out:
/* Print error for this request, and set parent request error message */
if (r->error != NULL && r->error[0] != '\0') {
warnx("%s", r->error);
gctl_error(req, "At least one provider failed to attach.");
}
if (r->error != NULL && r->error[0] != '\0') {
warnx("%s", r->error);
gctl_error(req, "There was an error with at least one provider.");
}
gctl_free(r);
/* Clear the derived key */
/* Clear sensitive data from memory. */
explicit_bzero(key, sizeof(key));
}
/* Clear the cached passphrase */
/* Clear sensitive data from memory. */
explicit_bzero(cached_passphrase, sizeof(cached_passphrase));
}