Skip to content

Commit 5f49c73

Browse files
committed
fix(v2): Adapt NewForwardingEntity to refactored NewEntity
1 parent 9aa010a commit 5f49c73

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

Diff for: openpgp/v2/forwarding.go

+4-4
Original file line numberDiff line numberDiff line change
@@ -26,14 +26,13 @@ func (e *Entity) NewForwardingEntity(
2626

2727
now := config.Now()
2828

29-
if _, err = e.VerifyPrimaryKey(now); err != nil {
29+
if _, err = e.VerifyPrimaryKey(now, config); err != nil {
3030
return nil, nil, err
3131
}
3232

3333
// Generate a new Primary key for the forwardee
3434
config.Algorithm = packet.PubKeyAlgoEdDSA
3535
config.Curve = packet.Curve25519
36-
keyLifetimeSecs := config.KeyLifetime()
3736

3837
forwardeePrimaryPrivRaw, err := newSigner(config)
3938
if err != nil {
@@ -49,7 +48,8 @@ func (e *Entity) NewForwardingEntity(
4948
Subkeys: []Subkey{},
5049
}
5150

52-
err = forwardeeKey.addUserId(userIdData{name, comment, email}, config, now, keyLifetimeSecs, true)
51+
keyProperties := selectKeyProperties(now, config, primary)
52+
err = forwardeeKey.addUserId(userIdData{name, comment, email}, config, keyProperties)
5353
if err != nil {
5454
return nil, nil, err
5555
}
@@ -64,7 +64,7 @@ func (e *Entity) NewForwardingEntity(
6464
continue
6565
}
6666

67-
forwarderSubKeySelfSig, err := forwarderSubKey.Verify(now)
67+
forwarderSubKeySelfSig, err := forwarderSubKey.Verify(now, config)
6868
// Filter expiration & revokal
6969
if err != nil {
7070
continue

Diff for: openpgp/v2/keys.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -636,7 +636,7 @@ func (e *Entity) Serialize(w io.Writer) error {
636636
// public key packets contain no meaningful information and do not need
637637
// to be serialized.
638638
// Prevent public key export for forwarding keys, see forwarding section 4.1.
639-
subKeySelfSig, err := subkey.LatestValidBindingSignature(time.Time{})
639+
subKeySelfSig, err := subkey.LatestValidBindingSignature(time.Time{}, nil)
640640
if subkey.PublicKey.PubKeyAlgo == packet.ExperimentalPubKeyAlgoHMAC ||
641641
subkey.PublicKey.PubKeyAlgo == packet.ExperimentalPubKeyAlgoAEAD ||
642642
(err == nil && subKeySelfSig.FlagForward) {

0 commit comments

Comments
 (0)