Opened 3 years ago

Closed 2 years ago

#2140 closed defect (no response)

Android NPE i2ptunnel SSL client

Reported by: zzz Owned by: Meeh
Priority: minor Milestone: n/a
Component: apps/android Version: 0.9.32
Keywords: Cc: str4d
Parent Tickets: Sensitive: no


<i2pr> [Slack/str4d] zzz, I've spotted one NPE that might be relevant to you (has only happened three times in the last release cycle)
<i2pr> [Slack/str4d] SelfSignedGenerator?.generate(), line 172, call to cert.checkValidity()
<i2pr> [Slack/str4d] throws a NPE inside
<i2pr> [Slack/str4d] called from
<i2pr> [Slack/str4d] The traceback (can't paste, looking at this on my phone) is from saving a tunnel, which in background calls GeneralHelper?.updateTunnelConfig(), which triggers SSLClientUtil.createKeyStore
<i2pr> [Slack/str4d] (which then calls KeyStoreUtil?.createKeys, leading to the above NPE)
<zzz> str4d, that would only be if somebody enabled SSL on a client tunnel. just tried it on android and desktop, no problems
<zzz> and the desktop certificate looks fine with certtool
<zzz> public static native void ASN1_TIME_to_Calendar(long asn1TimeCtx, Calendar cal);
<zzz> so it's not working on the cert at all, seems unlikely to be our fault?


Change History (4)

comment:1 Changed 3 years ago by Meeh

Milestone: 0.9.340.9.35

comment:2 Changed 3 years ago by Meeh

Owner: changed from str4d to Meeh
Status: newassigned

comment:3 Changed 3 years ago by zzz

Cc: str4d added
Status: assignedinfoneeded

str4d or meeh, please paste the full traceback from 8 months ago

comment:4 Changed 2 years ago by zzz

Milestone: 0.9.35n/a
Resolution: no response
Status: infoneededclosed

SSL disabled in android, see #2296
so this ticket no longer relevant.

meeh reports that only 60 days of crash reports are available in gplay
In the future we need full stack traces for all gplay tickets immediately so this doesn't happen again

Note: See TracTickets for help on using tickets.