[Rpm-maint] [rpm-software-management/rpm] Implement a separate lock for the keystore (PR #4123)
Michal Domonkos
notifications at github.com
Wed Apr 22 07:57:25 UTC 2026
@dmnks commented on this pull request.
> @@ -370,15 +389,18 @@ rpmRC rpmtxnImportPubkey(rpmtxn txn, const unsigned char * pkt, size_t pktlen)
exit:
/* Clean up. */
+ if (txn != kxn)
+ rpmtxnEnd(txn);
> Oh, I misread this initially: `krlock` is a member of rpmts, not rpmtxn. I guess this could be seen as an argument for the ts_lock, txn_lock school of struct member naming.
Okay, that's another thing that I misread yesterday :sweat_smile: Dunno why I thought it was `ts->lock` in there. So, nevermind this one either.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/4123?email_source=notifications&email_token=ADLPZU3AGOODPDJSZ5QCDD34XB3OLA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIMJVGI4TINRTGYZ2M4TFMFZW63VKON2WE43DOJUWEZLEUVSXMZLOOS6XA4S7OJSXM2LFO5PW433UNFTGSY3BORUW63TTL5RWY2LDNM#discussion_r3122395503
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/4123/review/4152946363 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20260422/8ca7724f/attachment-0001.htm>
More information about the Rpm-maint
mailing list