diff options
author | Neal H. Walfield <neal@pep.foundation> | 2022-11-18 10:11:47 +0100 |
---|---|---|
committer | Neal H. Walfield <neal@pep.foundation> | 2022-11-18 10:11:47 +0100 |
commit | 551381ee99c4d76825dc370c662c632913bd9393 (patch) | |
tree | 1dc9a43e2fb8b62ce1adfb87c802334c39d41192 /openpgp | |
parent | ba35204e4848b49fd100c5cc5f8e0b1cf109b69d (diff) |
openpgp: Fix use of Error::PolicyViolation.
- When rejecting a bad critical notation, `Error::PolicyViolation`
was used incorrectly. The first field is the thing that is in
violation of the policy, not a description.
Diffstat (limited to 'openpgp')
-rw-r--r-- | openpgp/src/policy.rs | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/openpgp/src/policy.rs b/openpgp/src/policy.rs index 03d36b02..542fa64f 100644 --- a/openpgp/src/policy.rs +++ b/openpgp/src/policy.rs @@ -1489,9 +1489,11 @@ impl<'a> Policy for StandardPolicy<'a> { .context("Policy rejected critical signature subpacket")?; if let SubpacketValue::NotationData(n) = csp.value() { if ! self.good_critical_notations.contains(&n.name()) { - return Err(Error::PolicyViolation( - format!("Policy rejected critical notation {:?}", - n.name()), None).into()); + return Err(anyhow::Error::from( + Error::PolicyViolation( + format!("Critical notation {:?}", + n.name()), None)) + .context("Policy rejected critical notation")); } } } |