From: Felix Dörre <felix@dogcraft.de>
Date: Thu, 15 Sep 2016 07:50:37 +0000 (+0200)
Subject: fix: certlogin. There was a "toLower" needed instead of an "toUpper"
X-Git-Url: https://code.wpia.club/?p=gigi.git;a=commitdiff_plain;h=d13a1d8a8506438a1f6b27f96f41e0ecaf612852;hp=448a1ea6318a255b571524cafe8b85aee1290e7a

fix: certlogin. There was a "toLower" needed instead of an "toUpper"

Change-Id: Ie233b6e920ec486a7e59d100681e86856bc7485c
---

diff --git a/src/org/cacert/gigi/Gigi.java b/src/org/cacert/gigi/Gigi.java
index 23f10df9..7f6b3c18 100644
--- a/src/org/cacert/gigi/Gigi.java
+++ b/src/org/cacert/gigi/Gigi.java
@@ -361,7 +361,7 @@ public final class Gigi extends HttpServlet {
         if (clientSerial != null) {
             X509Certificate[] cert = (X509Certificate[]) req.getAttribute("javax.servlet.request.X509Certificate");
             if (cert == null || cert[0] == null//
-                    || !cert[0].getSerialNumber().toString(16).toUpperCase().equals(clientSerial) //
+                    || !cert[0].getSerialNumber().toString(16).toLowerCase().equals(clientSerial) //
                     || !cert[0].getIssuerDN().equals(hs.getAttribute(CERT_ISSUER))) {
                 hs.invalidate();
                 resp.sendError(403, "Certificate mismatch.");