Questionable / Incorrect Code

Joshua Rogers git at internot.info
Thu Dec 25 02:07:48 CET 2014


On 25/12/14 07:10, Werner Koch wrote:
> On Wed, 24 Dec 2014 14:41, git at internot.info said:
>
>> > I probably explained that terribly. I'm very tired. But if you look at
>> > line 2340 you'll see what I mean.
> Indeed that looks like a faulty merge.  We need to go back and see why
> it happened.
It was introduced in commit 7a7a59782766a8bde0c3e7156d14bb2b0e4a3951, by
Marcus Brinkmann, just for reference.


> 26b4a012 (NIIBE Yutaka     2011-11-28 16:16:38 +0900 2337)   xfree
> (pin_verify);
> 26b4a012 (NIIBE Yutaka     2011-11-28 16:16:38 +0900 2338)   if (sw ||
> resultlen < 2)
> 7a7a5978 (Marcus Brinkmann 2012-01-03 22:12:37 +0100 2339)     return
> sw? sw : SW_HOST_INCOMPLETE_CARD_RESPONSE;
> 7a7a5978 (Marcus Brinkmann 2012-01-03 22:12:37 +0100 2340)   sw =
> (result[resultlen-2] << 8) | result[resultlen-1];
> 07f20f31 (NIIBE Yutaka     2011-12-20 13:34:27 +0900 2341)     {
> 07f20f31 (NIIBE Yutaka     2011-12-20 13:34:27 +0900 2342)      
> log_error ("control_pcsc failed: %d\n", sw);


I see no reason as to why a faluty merge would have caused that, though.
It looks like a copy and paste gone stray to me, since the exact same
code it below that. It doesn't really matter, though.

The commit log is as followed:

> commit 7a7a59782766a8bde0c3e7156d14bb2b0e4a3951
> Author: Marcus Brinkmann <marcus.brinkmann at ruhr-uni-bochum.de>
> Date:   Tue Jan 3 22:12:37 2012 +0100
>
>     Port to npth.
>     
>     * configure.ac: Don't check for PTH but for NPTH.
>     (AH_BOTTOM): Remove PTH_SYSCALL_SOFT.
>     (have_pth): Rename to ...
>     (have_npth): ... this.
>     (USE_GNU_NPTH): Rename to ...
>     (USE_GNU_PTH): ... this.
>     * m4/npth.m4: New file.
>     * agent/Makefile.am, agent/cache.c, agent/call-pinentry.c,
>     agent/call-scd.c, agent/findkey.c, agent/gpg-agent.c,
>     agent/trustlist.c, common/Makefile.am, common/estream.c,
>     common/exechelp-posix.c, common/exechelp-w32.c,
>     common/exechelp-w32ce.c, common/http.c, common/init.c,
>     common/sysutils.c, dirmngr/Makefile.am, dirmngr/crlfetch.c,
>     dirmngr/dirmngr.c, dirmngr/dirmngr_ldap.c, dirmngr/ldap-wrapper-ce.c,
>     dirmngr/ldap-wrapper.c, dirmngr/ldap.c, g13/Makefile.am,
>     g13/call-gpg.c, g13/g13.c, g13/runner.c, scd/Makefile.am,
>     scd/apdu.c, scd/app.c, scd/ccid-driver.c, scd/command.c,
>     scd/scdaemon.c, tools/Makefile.am: Port to npth.



Thanks,
-- 
-- Joshua Rogers <https://internot.info/>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: </pipermail/attachments/20141225/960cf4c3/attachment.sig>


More information about the Gnupg-devel mailing list