<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/gpg-interface.c, branch v2.1.2</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.1.2</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.1.2'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2014-06-23T22:50:30Z</updated>
<entry>
<title>gpg-interface: provide access to the payload</title>
<updated>2014-06-23T22:50:30Z</updated>
<author>
<name>Michael J Gruber</name>
<email>git@drmicha.warpmail.net</email>
</author>
<published>2014-06-23T07:05:48Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=71c214c840782a67801fc8dbf5fe8a4f4fc62d01'/>
<id>urn:sha1:71c214c840782a67801fc8dbf5fe8a4f4fc62d01</id>
<content type='text'>
In contrast to tag signatures, commit signatures are put into the
header, that is between the other header parts and commit messages.

Provide access to the commit content sans the signature, which is the
payload that is actually signed. Commit signature verification does the
parsing anyways, and callers may wish to act on or display the commit
object sans the signature.

Signed-off-by: Michael J Gruber &lt;git@drmicha.warpmail.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>gpg-interface: provide clear helper for struct signature_check</title>
<updated>2014-06-23T22:50:29Z</updated>
<author>
<name>Michael J Gruber</name>
<email>git@drmicha.warpmail.net</email>
</author>
<published>2014-06-23T07:05:47Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=01e57b5d91b0c9f2ac93708c5c2cbcd4731ddd34'/>
<id>urn:sha1:01e57b5d91b0c9f2ac93708c5c2cbcd4731ddd34</id>
<content type='text'>
The struct has been growing members whose malloced memory needs to be
freed. Do this with one helper function so that no malloced memory shall
be left unfreed.

Signed-off-by: Michael J Gruber &lt;git@drmicha.warpmail.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'mg/gpg-interface-using-status'</title>
<updated>2013-03-21T21:02:55Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-03-21T21:02:55Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=0f6875dbe2598fd2e9a597a9e42cde4a507140dd'/>
<id>urn:sha1:0f6875dbe2598fd2e9a597a9e42cde4a507140dd</id>
<content type='text'>
Call "gpg" using the right API when validating the signature on
tags.

* mg/gpg-interface-using-status:
  pretty: make %GK output the signing key for signed commits
  pretty: parse the gpg status lines rather than the output
  gpg_interface: allow to request status return
  log-tree: rely upon the check in the gpg_interface
  gpg-interface: check good signature in a reliable way
</content>
</entry>
<entry>
<title>gpg_interface: allow to request status return</title>
<updated>2013-02-14T17:30:04Z</updated>
<author>
<name>Michael J Gruber</name>
<email>git@drmicha.warpmail.net</email>
</author>
<published>2013-02-14T16:04:44Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=9cc4ac8ff1ae84f9435f2c7de3f7ab796103adba'/>
<id>urn:sha1:9cc4ac8ff1ae84f9435f2c7de3f7ab796103adba</id>
<content type='text'>
Currently, verify_signed_buffer() returns the user facing output only.

Allow callers to request the status output also.

Signed-off-by: Michael J Gruber &lt;git@drmicha.warpmail.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>gpg-interface: check good signature in a reliable way</title>
<updated>2013-02-14T17:27:40Z</updated>
<author>
<name>Michael J Gruber</name>
<email>git@drmicha.warpmail.net</email>
</author>
<published>2013-02-14T16:04:42Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=b60b7566c04e5f54c0e40229c1716d99d834ab68'/>
<id>urn:sha1:b60b7566c04e5f54c0e40229c1716d99d834ab68</id>
<content type='text'>
Currently, verify_signed_buffer() only checks the return code of gpg,
and some callers implement additional unreliable checks for "Good
signature" in the gpg output meant for the user.

Use the status output instead and parse for a line beinning with
"[GNUPG:] GOODSIG ". This is the only reliable way of checking for a
good gpg signature.

If needed we can change this easily to "[GNUPG:] VALIDSIG " if we want
to take into account the trust model.

Signed-off-by: Michael J Gruber &lt;git@drmicha.warpmail.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'sb/gpg-plug-fd-leak' into maint</title>
<updated>2013-02-07T23:14:54Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-02-07T23:14:54Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=d2216a4b133f90a543a0e4c4a9f9809a043012d6'/>
<id>urn:sha1:d2216a4b133f90a543a0e4c4a9f9809a043012d6</id>
<content type='text'>
We forgot to close the file descriptor reading from "gpg" output,
killing "git log --show-signature" on a long history.

* sb/gpg-plug-fd-leak:
  gpg: close stderr once finished with it in verify_signed_buffer()
</content>
</entry>
<entry>
<title>Merge branch 'sb/gpg-i18n'</title>
<updated>2013-02-07T22:41:34Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-02-07T22:41:34Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=9a1ab9e72fe8acd03b92bb481d7cd95207fda8f5'/>
<id>urn:sha1:9a1ab9e72fe8acd03b92bb481d7cd95207fda8f5</id>
<content type='text'>
* sb/gpg-i18n:
  gpg: allow translation of more error messages
</content>
</entry>
<entry>
<title>Merge branch 'sb/gpg-plug-fd-leak'</title>
<updated>2013-02-06T00:12:43Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-02-06T00:12:43Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=15778842bd5a63a2550c3d213b157823610a4f08'/>
<id>urn:sha1:15778842bd5a63a2550c3d213b157823610a4f08</id>
<content type='text'>
We forgot to close the file descriptor reading from "gpg" output,
killing "git log --show-signature" on a long history.

* sb/gpg-plug-fd-leak:
  gpg: close stderr once finished with it in verify_signed_buffer()
</content>
</entry>
<entry>
<title>gpg: close stderr once finished with it in verify_signed_buffer()</title>
<updated>2013-01-31T19:10:44Z</updated>
<author>
<name>Stephen Boyd</name>
<email>sboyd@codeaurora.org</email>
</author>
<published>2013-01-31T18:18:40Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=7dac3f83218b9f6c4c7d1c1cf093190f20d27108'/>
<id>urn:sha1:7dac3f83218b9f6c4c7d1c1cf093190f20d27108</id>
<content type='text'>
Failing to close the stderr pipe in verify_signed_buffer() causes
git to run out of file descriptors if there are many calls to
verify_signed_buffer(). An easy way to trigger this is to run

 git log --show-signature --merges | grep "key"

on the linux kernel git repo. Eventually it will fail with

 error: cannot create pipe for gpg: Too many open files
 error: could not run gpg.

Close the stderr pipe so that this can't happen.

Suggested-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>gpg: allow translation of more error messages</title>
<updated>2013-01-31T19:10:26Z</updated>
<author>
<name>Stephen Boyd</name>
<email>sboyd@codeaurora.org</email>
</author>
<published>2013-01-31T02:01:06Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=4c9a418227a55832de90ec2662570c9daef537da'/>
<id>urn:sha1:4c9a418227a55832de90ec2662570c9daef537da</id>
<content type='text'>
Mark these strings for translation so that error messages are
printed in the user's language of choice.

Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
Reviewed-by: Jonathan Nieder &lt;jrnieder@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
