#include "zypp/ZYppFactory.h"
#include "zypp/ZYpp.h"
-#include "zypp/base/Logger.h"
+#include "zypp/base/LogTools.h"
#include "zypp/base/IOStream.h"
#include "zypp/base/String.h"
#include "zypp/base/Regex.h"
+#include "zypp/base/Gettext.h"
+#include "zypp/base/WatchFile.h"
#include "zypp/PathInfo.h"
#include "zypp/KeyRing.h"
#include "zypp/ExternalProgram.h"
#include "zypp/TmpPath.h"
-using namespace std;
-using namespace zypp::filesystem;
+using std::endl;
#undef ZYPP_BASE_LOGGER_LOGGROUP
#define ZYPP_BASE_LOGGER_LOGGROUP "zypp::KeyRing"
+/** \todo Fix duplicate define in PublicKey/KeyRing */
#define GPG_BINARY "/usr/bin/gpg2"
///////////////////////////////////////////////////////////////////
IMPL_PTR_TYPE(KeyRing);
- static bool printLine( const string &line )
+ namespace
{
- MIL << line << endl;
- return true;
+ KeyRing::DefaultAccept _keyRingDefaultAccept( KeyRing::ACCEPT_NOTHING );
}
- namespace
+ KeyRing::DefaultAccept KeyRing::defaultAccept()
+ { return _keyRingDefaultAccept; }
+
+ void KeyRing::setDefaultAccept( DefaultAccept value_r )
{
- bool _keyRingDefaultAccept( getenv("ZYPP_KEYRING_DEFAULT_ACCEPT_ALL") );
+ MIL << "Set new KeyRing::DefaultAccept: " << value_r << endl;
+ _keyRingDefaultAccept = value_r;
}
- bool KeyRingReport::askUserToAcceptUnsignedFile( const string &file )
- { return _keyRingDefaultAccept; }
+ void KeyRingReport::infoVerify( const std::string & file_r, const PublicKeyData & keyData_r, const KeyContext & keycontext )
+ {}
- bool KeyRingReport::askUserToAcceptUnknownKey( const string &file, const string &id )
- { return _keyRingDefaultAccept; }
+ bool KeyRingReport::askUserToAcceptUnsignedFile( const std::string & file, const KeyContext & keycontext )
+ { return _keyRingDefaultAccept.testFlag( KeyRing::ACCEPT_UNSIGNED_FILE ); }
- bool KeyRingReport::askUserToTrustKey( const PublicKey &key )
- { return _keyRingDefaultAccept; }
+ KeyRingReport::KeyTrust
+ KeyRingReport::askUserToAcceptKey( const PublicKey & key, const KeyContext & keycontext )
+ {
+ if ( _keyRingDefaultAccept.testFlag( KeyRing::TRUST_KEY_TEMPORARILY ) )
+ return KEY_TRUST_TEMPORARILY;
+ if ( _keyRingDefaultAccept.testFlag( KeyRing::TRUST_AND_IMPORT_KEY ) )
+ return KEY_TRUST_AND_IMPORT;
+ return KEY_DONT_TRUST;
+ }
+
+ bool KeyRingReport::askUserToAcceptUnknownKey( const std::string & file, const std::string & id, const KeyContext & keycontext )
+ { return _keyRingDefaultAccept.testFlag( KeyRing::ACCEPT_UNKNOWNKEY ); }
+
+ bool KeyRingReport::askUserToAcceptVerificationFailed( const std::string & file, const PublicKey & key, const KeyContext & keycontext )
+ { return _keyRingDefaultAccept.testFlag( KeyRing::ACCEPT_VERIFICATION_FAILED ); }
+
+ namespace
+ {
+ ///////////////////////////////////////////////////////////////////
+ /// \class CachedPublicKeyData
+ /// \brief Functor returning the keyrings data (cached).
+ /// \code
+ /// const std::list<PublicKeyData> & cachedPublicKeyData( const Pathname & keyring );
+ /// \endcode
+ ///////////////////////////////////////////////////////////////////
+ struct CachedPublicKeyData // : private base::NonCopyable - but KeyRing uses RWCOW though also NonCopyable :(
+ {
+ const std::list<PublicKeyData> & operator()( const Pathname & keyring_r ) const
+ { return getData( keyring_r ); }
+
+ private:
+ struct Cache
+ {
+ // Empty copy ctor to allow insert into std::map as
+ // scoped_ptr is noncopyable.
+ Cache() {}
+ Cache( const Cache & rhs ) {}
+
+ void assertCache( const Pathname & keyring_r )
+ {
+ // .kbx since gpg2-2.1
+ if ( !_keyringK )
+ _keyringK.reset( new WatchFile( keyring_r/"pubring.kbx", WatchFile::NO_INIT ) );
+ if ( !_keyringP )
+ _keyringP.reset( new WatchFile( keyring_r/"pubring.gpg", WatchFile::NO_INIT ) );
+ }
+
+ bool hasChanged() const
+ {
+ bool k = _keyringK->hasChanged(); // be sure both files are checked
+ bool p = _keyringP->hasChanged();
+ return k || p;
+ }
+
+ std::list<PublicKeyData> _data;
+
+ private:
+ scoped_ptr<WatchFile> _keyringK;
+ scoped_ptr<WatchFile> _keyringP;
+ };
+
+ typedef std::map<Pathname,Cache> CacheMap;
+
+ const std::list<PublicKeyData> & getData( const Pathname & keyring_r ) const
+ {
+ Cache & cache( _cacheMap[keyring_r] );
+ // init new cache entry
+ cache.assertCache( keyring_r );
+ return getData( keyring_r, cache );
+ }
+
+ const std::list<PublicKeyData> & getData( const Pathname & keyring_r, Cache & cache_r ) const
+ {
+ if ( cache_r.hasChanged() )
+ {
+ const char* argv[] =
+ {
+ GPG_BINARY,
+ "--list-public-keys",
+ "--homedir", keyring_r.c_str(),
+ "--no-default-keyring",
+ "--quiet",
+ "--with-colons",
+ "--fixed-list-mode",
+ "--with-fingerprint",
+ "--with-sig-list",
+ "--no-tty",
+ "--no-greeting",
+ "--batch",
+ "--status-fd", "1",
+ NULL
+ };
+
+ PublicKeyScanner scanner;
+ ExternalProgram prog( argv ,ExternalProgram::Discard_Stderr, false, -1, true );
+ for( std::string line = prog.receiveLine(); !line.empty(); line = prog.receiveLine() )
+ {
+ scanner.scan( line );
+ }
+ prog.close();
+
+ cache_r._data.swap( scanner._keys );
+ MIL << "Found keys: " << cache_r._data << endl;
+ }
+ return cache_r._data;
+ }
+
+ mutable CacheMap _cacheMap;
+ };
+ ///////////////////////////////////////////////////////////////////
+ }
- bool KeyRingReport::askUserToImportKey( const PublicKey &key)
- { return _keyRingDefaultAccept; }
-
- bool KeyRingReport::askUserToAcceptVerificationFailed( const string &file, const PublicKey &key )
- { return _keyRingDefaultAccept; }
-
///////////////////////////////////////////////////////////////////
//
// CLASS NAME : KeyRing::Impl
/** KeyRing implementation. */
struct KeyRing::Impl
{
- Impl(const Pathname &baseTmpDir)
- : _trusted_tmp_dir(baseTmpDir, "zypp-trusted-kr")
- , _general_tmp_dir(baseTmpDir, "zypp-general-kr")
- , _base_dir( baseTmpDir )
-
+ Impl( const Pathname & baseTmpDir )
+ : _trusted_tmp_dir( baseTmpDir, "zypp-trusted-kr" )
+ , _general_tmp_dir( baseTmpDir, "zypp-general-kr" )
+ , _base_dir( baseTmpDir )
{
+ MIL << "Current KeyRing::DefaultAccept: " << _keyRingDefaultAccept << endl;
}
- /*
- Impl( const Pathname &general_kr, const Pathname &trusted_kr )
- {
- filesystem::assert_dir(general_kr);
- filesystem::assert_dir(trusted_kr);
+ void importKey( const PublicKey & key, bool trusted = false );
+ void multiKeyImport( const Pathname & keyfile_r, bool trusted_r = false );
+ void deleteKey( const std::string & id, bool trusted );
- generalKeyRing() = general_kr;
- trustedKeyRing() = trusted_kr;
- }
- */
+ std::string readSignatureKeyId( const Pathname & signature );
+
+ bool isKeyTrusted( const std::string & id )
+ { return bool(publicKeyExists( id, trustedKeyRing() )); }
+ bool isKeyKnown( const std::string & id )
+ { return publicKeyExists( id, trustedKeyRing() ) || publicKeyExists( id, generalKeyRing() ); }
+
+ std::list<PublicKey> trustedPublicKeys()
+ { return publicKeys( trustedKeyRing() ); }
+ std::list<PublicKey> publicKeys()
+ { return publicKeys( generalKeyRing() ); }
- void importKey( const PublicKey &key, bool trusted = false);
- void deleteKey( const string &id, bool trusted );
-
- string readSignatureKeyId( const Pathname &signature );
-
- bool isKeyTrusted( const string &id);
- bool isKeyKnown( const string &id );
-
- list<PublicKey> trustedPublicKeys();
- list<PublicKey> publicKeys();
+ const std::list<PublicKeyData> & trustedPublicKeyData()
+ { return publicKeyData( trustedKeyRing() ); }
+ const std::list<PublicKeyData> & publicKeyData()
+ { return publicKeyData( generalKeyRing() ); }
- list<string> trustedPublicKeyIds();
- list<string> publicKeyIds();
+ void dumpPublicKey( const std::string & id, bool trusted, std::ostream & stream )
+ { dumpPublicKey( id, ( trusted ? trustedKeyRing() : generalKeyRing() ), stream ); }
- void dumpPublicKey( const string &id, bool trusted, ostream &stream );
+ PublicKey exportPublicKey( const PublicKeyData & keyData )
+ { return exportKey( keyData, generalKeyRing() ); }
+ PublicKey exportTrustedPublicKey( const PublicKeyData & keyData )
+ { return exportKey( keyData, trustedKeyRing() ); }
- bool verifyFileSignatureWorkflow( const Pathname &file, const string filedesc, const Pathname &signature);
+ bool verifyFileSignatureWorkflow( const Pathname & file, const std::string & filedesc, const Pathname & signature, bool & sigValid_r, const KeyContext & keycontext = KeyContext());
+
+ bool verifyFileSignature( const Pathname & file, const Pathname & signature )
+ { return verifyFile( file, signature, generalKeyRing() ); }
+ bool verifyFileTrustedSignature( const Pathname & file, const Pathname & signature )
+ { return verifyFile( file, signature, trustedKeyRing() ); }
- bool verifyFileSignature( const Pathname &file, const Pathname &signature);
- bool verifyFileTrustedSignature( const Pathname &file, const Pathname &signature);
private:
- //mutable map<Locale, string> translations;
- bool verifyFile( const Pathname &file, const Pathname &signature, const Pathname &keyring);
- void importKey( const Pathname &keyfile, const Pathname &keyring);
- PublicKey exportKey( string id, const Pathname &keyring);
- void dumpPublicKey( const string &id, const Pathname &keyring, ostream &stream );
- void deleteKey( const string &id, const Pathname &keyring );
-
- list<PublicKey> publicKeys(const Pathname &keyring);
- list<string> publicKeyIds(const Pathname &keyring);
-
- bool publicKeyExists( string id, const Pathname &keyring);
-
- const Pathname generalKeyRing() const;
- const Pathname trustedKeyRing() const;
+ bool verifyFile( const Pathname & file, const Pathname & signature, const Pathname & keyring );
+ void importKey( const Pathname & keyfile, const Pathname & keyring );
+
+ PublicKey exportKey( const std::string & id, const Pathname & keyring );
+ PublicKey exportKey( const PublicKeyData & keyData, const Pathname & keyring );
+
+ void dumpPublicKey( const std::string & id, const Pathname & keyring, std::ostream & stream );
+ filesystem::TmpFile dumpPublicKeyToTmp( const std::string & id, const Pathname & keyring );
+
+ void deleteKey( const std::string & id, const Pathname & keyring );
+
+ std::list<PublicKey> publicKeys( const Pathname & keyring);
+ const std::list<PublicKeyData> & publicKeyData( const Pathname & keyring )
+ { return cachedPublicKeyData( keyring ); }
+
+ /** Get \ref PublicKeyData for ID (\c false if ID is not found). */
+ PublicKeyData publicKeyExists( const std::string & id, const Pathname & keyring );
+
+ const Pathname generalKeyRing() const
+ { return _general_tmp_dir.path(); }
+ const Pathname trustedKeyRing() const
+ { return _trusted_tmp_dir.path(); }
// Used for trusted and untrusted keyrings
- TmpDir _trusted_tmp_dir;
- TmpDir _general_tmp_dir;
+ filesystem::TmpDir _trusted_tmp_dir;
+ filesystem::TmpDir _general_tmp_dir;
Pathname _base_dir;
- public:
- /** Offer default Impl. */
- static shared_ptr<Impl> nullimpl()
- {
- static shared_ptr<Impl> _nullimpl( new Impl( TmpPath::defaultLocation() ) );
- return _nullimpl;
- }
private:
- friend Impl * rwcowClone<Impl>( const Impl * rhs );
- /** clone for RWCOW_pointer */
- Impl * clone() const
- { return new Impl( *this ); }
+ /** Functor returning the keyrings data (cached).
+ * \code
+ * const std::list<PublicKeyData> & cachedPublicKeyData( const Pathname & keyring );
+ * \endcode
+ */
+ CachedPublicKeyData cachedPublicKeyData;
};
+ ///////////////////////////////////////////////////////////////////
- const Pathname KeyRing::Impl::generalKeyRing() const
- {
- return _general_tmp_dir.path();
- }
-
- const Pathname KeyRing::Impl::trustedKeyRing() const
- {
- return _trusted_tmp_dir.path();
- }
-
- void KeyRing::Impl::importKey( const PublicKey &key, bool trusted)
+ void KeyRing::Impl::importKey( const PublicKey & key, bool trusted )
{
- callback::SendReport<KeyRingSignals> emitSignal;
-
importKey( key.path(), trusted ? trustedKeyRing() : generalKeyRing() );
-
+
if ( trusted )
+ {
+ callback::SendReport<target::rpm::KeyRingSignals> rpmdbEmitSignal;
+ callback::SendReport<KeyRingSignals> emitSignal;
+
+ rpmdbEmitSignal->trustedKeyAdded( key );
emitSignal->trustedKeyAdded( key );
-
+ }
}
- void KeyRing::Impl::deleteKey( const string &id, bool trusted)
+ void KeyRing::Impl::multiKeyImport( const Pathname & keyfile_r, bool trusted_r )
{
- deleteKey( id, trusted ? trustedKeyRing() : generalKeyRing() );
+ importKey( keyfile_r, trusted_r ? trustedKeyRing() : generalKeyRing() );
}
- list<PublicKey> KeyRing::Impl::publicKeys()
+ void KeyRing::Impl::deleteKey( const std::string & id, bool trusted )
{
- return publicKeys( generalKeyRing() );
- }
+ PublicKey key;
- list<PublicKey> KeyRing::Impl::trustedPublicKeys()
- {
- return publicKeys( trustedKeyRing() );
- }
+ if ( trusted )
+ {
+ key = exportKey( id, trustedKeyRing() );
+ }
- list<string> KeyRing::Impl::publicKeyIds()
- {
- return publicKeyIds( generalKeyRing() );
- }
+ deleteKey( id, trusted ? trustedKeyRing() : generalKeyRing() );
- list<string> KeyRing::Impl::trustedPublicKeyIds()
- {
- return publicKeyIds( trustedKeyRing() );
- }
-
- bool KeyRing::Impl::verifyFileTrustedSignature( const Pathname &file, const Pathname &signature)
- {
- return verifyFile( file, signature, trustedKeyRing() );
- }
+ if ( trusted )
+ {
+ callback::SendReport<target::rpm::KeyRingSignals> rpmdbEmitSignal;
+ callback::SendReport<KeyRingSignals> emitSignal;
- bool KeyRing::Impl::verifyFileSignature( const Pathname &file, const Pathname &signature)
- {
- return verifyFile( file, signature, generalKeyRing() );
+ rpmdbEmitSignal->trustedKeyRemoved( key );
+ emitSignal->trustedKeyRemoved( key );
+ }
}
- bool KeyRing::Impl::isKeyTrusted( const string &id)
- {
- return publicKeyExists( id, trustedKeyRing() );
- }
-
- bool KeyRing::Impl::isKeyKnown( const string &id )
- {
- MIL << endl;
- if ( publicKeyExists( id, trustedKeyRing() ) )
- return true;
- else
- return publicKeyExists( id, generalKeyRing() );
- }
-
- bool KeyRing::Impl::publicKeyExists( string id, const Pathname &keyring)
+ PublicKeyData KeyRing::Impl::publicKeyExists( const std::string & id, const Pathname & keyring )
{
MIL << "Searching key [" << id << "] in keyring " << keyring << endl;
- list<PublicKey> keys = publicKeys(keyring);
- for (list<PublicKey>::const_iterator it = keys.begin(); it != keys.end(); it++)
+ const std::list<PublicKeyData> & keys( publicKeyData( keyring ) );
+ for_( it, keys.begin(), keys.end() )
{
if ( id == (*it).id() )
- return true;
+ {
+ return *it;
+ }
}
- return false;
+ return PublicKeyData();
}
-
- PublicKey KeyRing::Impl::exportKey( string id, const Pathname &keyring)
+
+ PublicKey KeyRing::Impl::exportKey( const PublicKeyData & keyData, const Pathname & keyring )
{
- TmpFile tmp_file( _base_dir, "pubkey-"+id+"-" );
- Pathname keyfile = tmp_file.path();
- MIL << "Going to export key " << id << " from " << keyring << " to " << keyfile << endl;
-
- try {
- ofstream os(keyfile.asString().c_str());
- dumpPublicKey( id, keyring, os );
- os.close();
- PublicKey key(keyfile);
- return key;
- }
- catch (BadKeyException &e)
- {
- ERR << "Cannot create public key " << id << " from " << keyring << " keyring to file " << e.keyFile() << endl;
- ZYPP_THROW(Exception("Cannot create public key " + id + " from " + keyring.asString() + " keyring to file " + e.keyFile().asString() ) );
- }
- catch (exception &e)
- {
- ERR << "Cannot export key " << id << " from " << keyring << " keyring to file " << keyfile << endl;
- }
- return PublicKey();
+ return PublicKey( dumpPublicKeyToTmp( keyData.id(), keyring ), keyData );
}
- void KeyRing::Impl::dumpPublicKey( const string &id, bool trusted, ostream &stream )
+ PublicKey KeyRing::Impl::exportKey( const std::string & id, const Pathname & keyring )
{
- dumpPublicKey( id, ( trusted ? trustedKeyRing() : generalKeyRing() ), stream );
+ PublicKeyData keyData( publicKeyExists( id, keyring ) );
+ if ( keyData )
+ return PublicKey( dumpPublicKeyToTmp( keyData.id(), keyring ), keyData );
+
+ // Here: key not found
+ WAR << "No key " << id << " to export from " << keyring << endl;
+ return PublicKey();
}
-
- void KeyRing::Impl::dumpPublicKey( const string &id, const Pathname &keyring, ostream &stream )
+
+
+ void KeyRing::Impl::dumpPublicKey( const std::string & id, const Pathname & keyring, std::ostream & stream )
{
const char* argv[] =
{
GPG_BINARY,
+ "-a",
+ "--export",
+ "--homedir", keyring.asString().c_str(),
"--no-default-keyring",
"--quiet",
"--no-tty",
"--no-greeting",
"--no-permission-warning",
"--batch",
- "--homedir",
- keyring.asString().c_str(),
- "-a",
- "--export",
id.c_str(),
NULL
};
- ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
- string line;
- int count;
- for(line = prog.receiveLine(), count=0; !line.empty(); line = prog.receiveLine(), count++ )
+ ExternalProgram prog( argv,ExternalProgram::Discard_Stderr, false, -1, true );
+ for ( std::string line = prog.receiveLine(); !line.empty(); line = prog.receiveLine() )
{
stream << line;
}
prog.close();
}
+ filesystem::TmpFile KeyRing::Impl::dumpPublicKeyToTmp( const std::string & id, const Pathname & keyring )
+ {
+ filesystem::TmpFile tmpFile( _base_dir, "pubkey-"+id+"-" );
+ MIL << "Going to export key " << id << " from " << keyring << " to " << tmpFile.path() << endl;
+
+ std::ofstream os( tmpFile.path().c_str() );
+ dumpPublicKey( id, keyring, os );
+ os.close();
+ return tmpFile;
+ }
- bool KeyRing::Impl::verifyFileSignatureWorkflow( const Pathname &file, const string filedesc, const Pathname &signature)
+ bool KeyRing::Impl::verifyFileSignatureWorkflow( const Pathname & file, const std::string & filedesc, const Pathname & signature, bool & sigValid_r, const KeyContext & context )
{
+ sigValid_r = false; // set true if signature is actually successfully validated!
+
callback::SendReport<KeyRingReport> report;
- //callback::SendReport<KeyRingSignals> emitSignal;
- MIL << "Going to verify signature for " << file << " with " << signature << endl;
+ MIL << "Going to verify signature for " << filedesc << " ( " << file << " ) with " << signature << endl;
// if signature does not exists, ask user if he wants to accept unsigned file.
- if( signature.empty() || (!PathInfo(signature).isExist()) )
+ if( signature.empty() || (!PathInfo( signature ).isExist()) )
{
- bool res = report->askUserToAcceptUnsignedFile( filedesc );
+ bool res = report->askUserToAcceptUnsignedFile( filedesc, context );
MIL << "User decision on unsigned file: " << res << endl;
return res;
}
// get the id of the signature
- string id = readSignatureKeyId(signature);
+ std::string id = readSignatureKeyId( signature );
// doeskey exists in trusted keyring
- if ( publicKeyExists( id, trustedKeyRing() ) )
+ PublicKeyData trustedKeyData( publicKeyExists( id, trustedKeyRing() ) );
+ if ( trustedKeyData )
{
- PublicKey key = exportKey( id, trustedKeyRing() );
-
- MIL << "Key " << id << " " << key.name() << " is trusted" << endl;
+ MIL << "Key is trusted: " << trustedKeyData << endl;
+
+ // lets look if there is an updated key in the
+ // general keyring
+ PublicKeyData generalKeyData( publicKeyExists( id, generalKeyRing() ) );
+ if ( generalKeyData )
+ {
+ // bnc #393160: Comment #30: Compare at least the fingerprint
+ // in case an attacker created a key the the same id.
+ if ( trustedKeyData.fingerprint() == generalKeyData.fingerprint()
+ && trustedKeyData.created() < generalKeyData.created() )
+ {
+ MIL << "Key was updated. Saving new version into trusted keyring: " << generalKeyData << endl;
+ importKey( exportKey( generalKeyData, generalKeyRing() ), true );
+ trustedKeyData = generalKeyData = PublicKeyData(); // invalidated by import.
+ }
+ }
+
+ if ( ! trustedKeyData ) // invalidated by previous import
+ trustedKeyData = publicKeyExists( id, trustedKeyRing() );
+ report->infoVerify( filedesc, trustedKeyData, context );
+
// it exists, is trusted, does it validates?
if ( verifyFile( file, signature, trustedKeyRing() ) )
- return true;
+ {
+ return (sigValid_r=true); // signature is actually successfully validated!
+ }
else
- return report->askUserToAcceptVerificationFailed( filedesc, key );
+ {
+ return report->askUserToAcceptVerificationFailed( filedesc, exportKey( trustedKeyData, trustedKeyRing() ), context );
+ }
}
else
{
- if ( publicKeyExists( id, generalKeyRing() ) )
+ PublicKeyData generalKeyData( publicKeyExists( id, generalKeyRing() ) );
+ if ( generalKeyData )
{
- PublicKey key = exportKey( id, generalKeyRing());
+ PublicKey key( exportKey( generalKeyData, generalKeyRing() ) );
MIL << "Exported key " << id << " to " << key.path() << endl;
MIL << "Key " << id << " " << key.name() << " is not trusted" << endl;
+
// ok the key is not trusted, ask the user to trust it or not
- #warning We need the key details passed to the callback
- if ( report->askUserToTrustKey( key ) )
+ KeyRingReport::KeyTrust reply = report->askUserToAcceptKey( key, context );
+ if ( reply == KeyRingReport::KEY_TRUST_TEMPORARILY ||
+ reply == KeyRingReport::KEY_TRUST_AND_IMPORT )
{
MIL << "User wants to trust key " << id << " " << key.name() << endl;
- //dumpFile(unKey.path());
+ //dumpFile( unKey.path() );
- Pathname which_keyring;
- if ( report->askUserToImportKey( key ) )
+ Pathname whichKeyring;
+ if ( reply == KeyRingReport::KEY_TRUST_AND_IMPORT )
{
MIL << "User wants to import key " << id << " " << key.name() << endl;
importKey( key, true );
- which_keyring = trustedKeyRing();
+ whichKeyring = trustedKeyRing();
}
else
- {
- which_keyring = generalKeyRing();
- }
+ whichKeyring = generalKeyRing();
// emit key added
- if ( verifyFile( file, signature, which_keyring ) )
+ if ( verifyFile( file, signature, whichKeyring ) )
{
MIL << "File signature is verified" << endl;
- return true;
+ return (sigValid_r=true); // signature is actually successfully validated!
}
else
{
MIL << "File signature check fails" << endl;
- if ( report->askUserToAcceptVerificationFailed( filedesc, key ) )
+ if ( report->askUserToAcceptVerificationFailed( filedesc, key, context ) )
{
MIL << "User continues anyway." << endl;
return true;
{
// unknown key...
MIL << "File [" << file << "] ( " << filedesc << " ) signed with unknown key [" << id << "]" << endl;
- if ( report->askUserToAcceptUnknownKey( filedesc, id ) )
+ if ( report->askUserToAcceptUnknownKey( filedesc, id, context ) )
{
MIL << "User wants to accept unknown key " << id << endl;
return true;
return false;
}
- list<string> KeyRing::Impl::publicKeyIds(const Pathname &keyring)
+ std::list<PublicKey> KeyRing::Impl::publicKeys( const Pathname & keyring )
{
- static str::regex rxColons("^([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):\n$");
- static str::regex rxColonsFpr("^([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):\n$");
-
- list<string> ids;
-
- const char* argv[] =
- {
- GPG_BINARY,
- "--no-default-keyring",
- "--quiet",
- "--list-public-keys",
- "--with-colons",
- "--with-fingerprint",
- "--no-tty",
- "--no-greeting",
- "--batch",
- "--status-fd",
- "1",
- "--homedir",
- keyring.asString().c_str(),
- NULL
- };
-
- ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
- string line;
- int count = 0;
+ const std::list<PublicKeyData> & keys( publicKeyData( keyring ) );
+ std::list<PublicKey> ret;
- for(line = prog.receiveLine(), count=0; !line.empty(); line = prog.receiveLine(), count++ )
+ for_( it, keys.begin(), keys.end() )
{
- //MIL << line << endl;
- str::smatch what;
- if(str::regex_match(line, what, rxColons))
- {
- string id;
- string fingerprint;
- if ( what[1] == "pub" )
- {
- id = what[5];
-
- string line2;
- for(line2 = prog.receiveLine(); !line2.empty(); line2 = prog.receiveLine(), count++ )
- {
- str::smatch what2;
- if (str::regex_match(line2, what2, rxColonsFpr))
- {
- if ( (what2[1] == "fpr") && (what2[1] != "pub") && (what2[1] !="sub"))
- {
- fingerprint = what2[10];
- break;
- }
- }
- }
-
- ids.push_back(id);
- MIL << "Found key " << "[" << id << "]" << endl;
- }
- //dumpRegexpResults(what);
- }
+ PublicKey key( exportKey( *it, keyring ) );
+ ret.push_back( key );
+ MIL << "Found key " << key << endl;
}
- prog.close();
- return ids;
+ return ret;
}
-
- list<PublicKey> KeyRing::Impl::publicKeys(const Pathname &keyring)
- {
-
- list<PublicKey> keys;
-
- list<string> ids = publicKeyIds(keyring);
-
- for ( list<string>::const_iterator it = ids.begin(); it != ids.end(); ++it )
- {
- PublicKey key(exportKey( *it, keyring ));
- keys.push_back(key);
- MIL << "Found key " << "[" << key.id() << "]" << " [" << key.name() << "]" << " [" << key.fingerprint() << "]" << endl;
- }
- return keys;
- }
-
- void KeyRing::Impl::importKey( const Pathname &keyfile, const Pathname &keyring)
+
+ void KeyRing::Impl::importKey( const Pathname & keyfile, const Pathname & keyring )
{
- if ( ! PathInfo(keyfile).isExist() )
- ZYPP_THROW(KeyRingException("Tried to import not existant key " + keyfile.asString() + " into keyring " + keyring.asString()));
-
+ if ( ! PathInfo( keyfile ).isExist() )
+ // TranslatorExplanation first %s is key name, second is keyring name
+ ZYPP_THROW(KeyRingException( str::Format(_("Tried to import not existent key %s into keyring %s"))
+ % keyfile.asString()
+ % keyring.asString() ));
+
const char* argv[] =
{
GPG_BINARY,
+ "--import",
+ "--homedir", keyring.asString().c_str(),
"--no-default-keyring",
"--quiet",
"--no-tty",
"--no-greeting",
"--no-permission-warning",
- "--status-fd",
- "1",
- "--homedir",
- keyring.asString().c_str(),
- "--import",
+ "--status-fd", "1",
keyfile.asString().c_str(),
NULL
};
- int code;
- ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
- code = prog.close();
-
- //if ( code != 0 )
- // ZYPP_THROW(Exception("failed to import key"));
+ ExternalProgram prog( argv,ExternalProgram::Discard_Stderr, false, -1, true );
+ prog.close();
}
- void KeyRing::Impl::deleteKey( const string &id, const Pathname &keyring )
+ void KeyRing::Impl::deleteKey( const std::string & id, const Pathname & keyring )
{
const char* argv[] =
{
GPG_BINARY,
+ "--delete-keys",
+ "--homedir", keyring.asString().c_str(),
"--no-default-keyring",
"--yes",
"--quiet",
"--no-tty",
"--batch",
- "--status-fd",
- "1",
- "--homedir",
- keyring.asString().c_str(),
- "--delete-keys",
+ "--status-fd", "1",
id.c_str(),
NULL
};
- ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
+ ExternalProgram prog( argv,ExternalProgram::Discard_Stderr, false, -1, true );
int code = prog.close();
if ( code )
- ZYPP_THROW(Exception("Failed to delete key."));
+ ZYPP_THROW(Exception(_("Failed to delete key.")));
else
MIL << "Deleted key " << id << " from keyring " << keyring << endl;
}
- string KeyRing::Impl::readSignatureKeyId(const Pathname &signature )
+ std::string KeyRing::Impl::readSignatureKeyId( const Pathname & signature )
{
- MIL << "Deetermining key id if signature " << signature << endl;
+ if ( ! PathInfo( signature ).isFile() )
+ ZYPP_THROW(Exception( str::Format(_("Signature file %s not found")) % signature.asString() ));
+
+ MIL << "Determining key id if signature " << signature << endl;
// HACK create a tmp keyring with no keys
- TmpDir dir(_base_dir, "fake-keyring");
- TmpFile fakeData(_base_dir, "fake-data");
+ filesystem::TmpDir dir( _base_dir, "fake-keyring" );
const char* argv[] =
{
GPG_BINARY,
+ "--homedir", dir.path().asString().c_str(),
"--no-default-keyring",
"--quiet",
"--no-tty",
"--no-greeting",
"--batch",
- "--status-fd",
- "1",
- "--homedir",
- dir.path().asString().c_str(),
- "--verify",
+ "--status-fd", "1",
signature.asString().c_str(),
- fakeData.path().asString().c_str(),
NULL
};
- ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
+ ExternalProgram prog( argv,ExternalProgram::Discard_Stderr, false, -1, true );
- string line;
+ std::string line;
int count = 0;
- str::regex rxNoKey("^\\[GNUPG:\\] NO_PUBKEY (.+)\n$");
- string id;
- for(line = prog.receiveLine(), count=0; !line.empty(); line = prog.receiveLine(), count++ )
+ str::regex rxNoKey( "^\\[GNUPG:\\] NO_PUBKEY (.+)\n$" );
+ std::string id;
+ for( line = prog.receiveLine(), count=0; !line.empty(); line = prog.receiveLine(), count++ )
{
//MIL << "[" << line << "]" << endl;
str::smatch what;
- if(str::regex_search(line, what, rxNoKey))
+ if( str::regex_match( line, what, rxNoKey ) )
{
- if ( what.size() > 1 )
+ if ( what.size() >= 1 )
+ {
id = what[1];
- //dumpRegexpResults(what);
+ break;
+ }
+ //dumpRegexpResults( what );
}
}
+
+ if ( count == 0 )
+ {
+ MIL << "no output" << endl;
+ }
+
MIL << "Determined key id [" << id << "] for signature " << signature << endl;
prog.close();
return id;
}
- bool KeyRing::Impl::verifyFile( const Pathname &file, const Pathname &signature, const Pathname &keyring)
+ bool KeyRing::Impl::verifyFile( const Pathname & file, const Pathname & signature, const Pathname & keyring )
{
const char* argv[] =
{
GPG_BINARY,
+ "--verify",
+ "--homedir", keyring.asString().c_str(),
"--no-default-keyring",
"--quiet",
"--no-tty",
"--batch",
"--no-greeting",
- "--status-fd",
- "1",
- "--homedir",
- keyring.asString().c_str(),
- "--verify",
+ "--status-fd", "1",
signature.asString().c_str(),
file.asString().c_str(),
NULL
// [GNUPG:] ERRSIG A84EDAE89C800ACA 17 2 00 1143618744 9
// [GNUPG:] NO_PUBKEY A84EDAE89C800ACA
- ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
+ ExternalProgram prog( argv,ExternalProgram::Discard_Stderr, false, -1, true );
- return (prog.close() == 0) ? true : false;
+ return ( prog.close() == 0 ) ? true : false;
}
///////////////////////////////////////////////////////////////////
//
///////////////////////////////////////////////////////////////////
- ///////////////////////////////////////////////////////////////////
- //
- // METHOD NAME : KeyRing::KeyRing
- // METHOD TYPE : Ctor
- //
- KeyRing::KeyRing(const Pathname &baseTmpDir)
- : _pimpl( new Impl(baseTmpDir) )
+ KeyRing::KeyRing( const Pathname & baseTmpDir )
+ : _pimpl( new Impl( baseTmpDir ) )
{}
- ///////////////////////////////////////////////////////////////////
- //
- // METHOD NAME : KeyRing::KeyRing
- // METHOD TYPE : Ctor
- //
- //KeyRing::KeyRing( const Pathname &general_kr, const Pathname &trusted_kr )
- //: _pimpl( new Impl(general_kr, trusted_kr) )
- //{}
-
- ///////////////////////////////////////////////////////////////////
- //
- // METHOD NAME : KeyRing::~KeyRing
- // METHOD TYPE : Dtor
- //
KeyRing::~KeyRing()
{}
- ///////////////////////////////////////////////////////////////////
- //
- // Forward to implementation:
- //
- ///////////////////////////////////////////////////////////////////
-
- void KeyRing::importKey( const PublicKey &key, bool trusted )
- {
- _pimpl->importKey( key.path(), trusted );
- }
-
- string KeyRing::readSignatureKeyId( const Pathname &signature )
- {
- return _pimpl->readSignatureKeyId(signature);
- }
+ void KeyRing::importKey( const PublicKey & key, bool trusted )
+ { _pimpl->importKey( key, trusted ); }
- void KeyRing::deleteKey( const string &id, bool trusted )
- {
- _pimpl->deleteKey(id, trusted);
- }
+ void KeyRing::multiKeyImport( const Pathname & keyfile_r, bool trusted_r )
+ { _pimpl->multiKeyImport( keyfile_r, trusted_r ); }
- list<PublicKey> KeyRing::publicKeys()
- {
- return _pimpl->publicKeys();
- }
+ std::string KeyRing::readSignatureKeyId( const Pathname & signature )
+ { return _pimpl->readSignatureKeyId( signature ); }
- list<PublicKey> KeyRing::trustedPublicKeys()
- {
- return _pimpl->trustedPublicKeys();
- }
+ void KeyRing::deleteKey( const std::string & id, bool trusted )
+ { _pimpl->deleteKey( id, trusted ); }
- list<string> KeyRing::publicKeyIds()
- {
- return _pimpl->publicKeyIds();
- }
+ std::list<PublicKey> KeyRing::publicKeys()
+ { return _pimpl->publicKeys(); }
- list<string> KeyRing::trustedPublicKeyIds()
- {
- return _pimpl->trustedPublicKeyIds();
- }
-
- bool KeyRing::verifyFileSignatureWorkflow( const Pathname &file, const string filedesc, const Pathname &signature)
- {
- return _pimpl->verifyFileSignatureWorkflow(file, filedesc, signature);
- }
+ std:: list<PublicKey> KeyRing::trustedPublicKeys()
+ { return _pimpl->trustedPublicKeys(); }
- bool KeyRing::verifyFileSignature( const Pathname &file, const Pathname &signature)
- {
- return _pimpl->verifyFileSignature(file, signature);
- }
+ std::list<PublicKeyData> KeyRing::publicKeyData()
+ { return _pimpl->publicKeyData(); }
- bool KeyRing::verifyFileTrustedSignature( const Pathname &file, const Pathname &signature)
- {
- return _pimpl->verifyFileTrustedSignature(file, signature);
- }
+ std::list<PublicKeyData> KeyRing::trustedPublicKeyData()
+ { return _pimpl->trustedPublicKeyData(); }
- void KeyRing::dumpPublicKey( const string &id, bool trusted, ostream &stream )
- {
- _pimpl->dumpPublicKey( id, trusted, stream);
- }
+ bool KeyRing::verifyFileSignatureWorkflow( const Pathname & file, const std::string & filedesc, const Pathname & signature, bool & sigValid_r, const KeyContext & keycontext )
+ { return _pimpl->verifyFileSignatureWorkflow( file, filedesc, signature, sigValid_r, keycontext ); }
+
+ bool KeyRing::verifyFileSignatureWorkflow( const Pathname & file, const std::string filedesc, const Pathname & signature, const KeyContext & keycontext )
+ { bool unused; return _pimpl->verifyFileSignatureWorkflow( file, filedesc, signature, unused, keycontext ); }
+
+ bool KeyRing::verifyFileSignature( const Pathname & file, const Pathname & signature )
+ { return _pimpl->verifyFileSignature( file, signature ); }
+
+ bool KeyRing::verifyFileTrustedSignature( const Pathname & file, const Pathname & signature )
+ { return _pimpl->verifyFileTrustedSignature( file, signature ); }
+
+ void KeyRing::dumpPublicKey( const std::string & id, bool trusted, std::ostream & stream )
+ { _pimpl->dumpPublicKey( id, trusted, stream ); }
+
+ PublicKey KeyRing::exportPublicKey( const PublicKeyData & keyData )
+ { return _pimpl->exportPublicKey( keyData ); }
+
+ PublicKey KeyRing::exportTrustedPublicKey( const PublicKeyData & keyData )
+ { return _pimpl->exportTrustedPublicKey( keyData ); }
+
+ bool KeyRing::isKeyTrusted( const std::string & id )
+ { return _pimpl->isKeyTrusted( id ); }
+
+ bool KeyRing::isKeyKnown( const std::string & id )
+ { return _pimpl->isKeyKnown( id ); }
- bool KeyRing::isKeyTrusted( const string &id )
- {
- return _pimpl->isKeyTrusted(id);
- }
-
- bool KeyRing::isKeyKnown( const string &id )
- {
- return _pimpl->isKeyKnown(id);
- }
-
/////////////////////////////////////////////////////////////////
} // namespace zypp
///////////////////////////////////////////////////////////////////