View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000814 | Main CAcert Website | GPG/PGP | public | 2010-03-28 23:20 | 2013-11-20 22:23 |
Reporter | Sourcerer | Assigned To | Sourcerer | ||
Priority | immediate | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Fixed in Version | 2010 Q1 | ||||
Summary | 0000814: Multiple OpenPGP keys are handled incorrectly | ||||
Description | When uploading an OpenPGP Base64 container that contains several keys, CAcert handles the keys wrongly, and prefers the first key. | ||||
Tags | No tags attached. | ||||
Attached Files | npubs.patch (910 bytes)
diff --git a/www/gpg.php b/www/gpg.php index f97966f..894f298 100644 --- a/www/gpg.php +++ b/www/gpg.php @@ -104,11 +104,25 @@ function verifyEmail($email) $resulttable=_("The following UIDs were found in your key:")."<br/><table border='1'><tr><td>#</td><td>"._("Name")."</td><td>"._("Email")."</td><td>Result</td>"; $i=0; $lastvalidemail=""; + $npubs=0; foreach(explode("\n", $gpg) as $line) { $bits = explode(":", $line); $resulttable.="<tr><td>".++$i."</td>"; $name = $comment = ""; + if($bits[0] == "pub") + { + $npubs++; + } + if($npubs>1) + { + showheader(_("Welcome to CAcert.org")); + echo "<font color='#ff0000'>"._("Please upload only one key at a time.")."</font>"; + unset($_REQUEST['process']); + $id = $oldid; + unset($oldid); + exit(); + } if($bits[0] == "pub" && (!$keyid || !$when)) { $keyid = $bits[4]; | ||||
Reviewed by | |||||
Test Instructions | |||||
Date Modified | Username | Field | Change |
---|---|---|---|
2010-03-28 23:20 | Sourcerer | New Issue | |
2010-03-28 23:20 | Sourcerer | File Added: npubs.patch | |
2010-03-29 09:40 | edgarwahn | Note Added: 0001569 | |
2010-03-29 18:21 | Sourcerer | Note Added: 0001572 | |
2010-03-29 18:21 | Sourcerer | Status | new => closed |
2010-03-29 18:21 | Sourcerer | Resolution | open => fixed |
2010-03-29 18:21 | Sourcerer | Assigned To | => Sourcerer |
2011-06-19 16:12 | edgarwahn | Source_changeset_attached | => cacert-devel release a84f4ce8 |
2011-06-19 16:46 | edgarwahn | Source_changeset_attached | => cacert-devel release a84f4ce8 |
2011-06-21 23:57 | edgarwahn | Source_changeset_attached | => cacert-devel release a84f4ce8 |
2011-06-21 23:57 | edgarwahn | Source_changeset_attached | => cacert-devel release 6c2e9fb5 |
2011-06-22 00:09 | edgarwahn | Source_changeset_attached | => cacert-devel release a84f4ce8 |
2011-06-22 00:09 | edgarwahn | Source_changeset_attached | => cacert-devel release 6c2e9fb5 |
2013-01-15 14:33 | Werner Dworak | Fixed in Version | => 2010 Q1 |
2013-11-20 22:23 | NEOatNHNG | View Status | private => public |