From 288c775034d4c18846518a11677e4580a91cf437 Mon Sep 17 00:00:00 2001 From: Ruben Pollan Date: Sun, 20 Mar 2016 18:44:23 +0100 Subject: [bug] Return KeyNotFound Failure if not valid key is given to put_raw_key - Resolves: #7974 --- changes/next-changelog.txt | 2 ++ src/leap/keymanager/__init__.py | 11 ++++++++--- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/changes/next-changelog.txt b/changes/next-changelog.txt index 163c1ee..2dd34d4 100644 --- a/changes/next-changelog.txt +++ b/changes/next-changelog.txt @@ -21,6 +21,8 @@ Features Bugfixes ~~~~~~~~ +- `#7974 `_: Return KeyNotFound Failure if not valid key is given to put_raw_key. + - `#1235 `_: Description for the fixed stuff corresponding with issue #1235. - Bugfix without related issue number. diff --git a/src/leap/keymanager/__init__.py b/src/leap/keymanager/__init__.py index 9e3b6ee..1106c23 100644 --- a/src/leap/keymanager/__init__.py +++ b/src/leap/keymanager/__init__.py @@ -820,9 +820,10 @@ class KeyManager(object): :return: A Deferred which fires when the key is in the storage, or which fails with KeyAddressMismatch if address doesn't match - any uid on the key or fails with KeyNotValidUpdate if a key - with the same uid exists and the new one is not a valid update - for it. + any uid on the key or fails with KeyNotFound if no OpenPGP + material was found in key or fails with KeyNotValidUpdate if a + key with the same uid exists and the new one is not a valid + update for it. :rtype: Deferred :raise UnsupportedKeyTypeError: if invalid key type @@ -831,6 +832,10 @@ class KeyManager(object): _keys = self._wrapper_map[ktype] pubkey, privkey = _keys.parse_ascii_key(key, address) + + if pubkey is None: + return defer.fail(KeyNotFound(key)) + pubkey.validation = validation d = self.put_key(pubkey) if privkey is not None: -- cgit v1.2.3