Opened 14 months ago

Last modified 6 months ago

#2140 infoneeded defect

Android NPE i2ptunnel SSL client

Reported by: zzz Owned by: meeh
Priority: minor Milestone: 0.9.35
Component: apps/android Version: 0.9.32
Keywords: Cc: str4d
Parent Tickets:

Description

<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 com.android.org.conscrypt.NativeCrypto?.ASN1_TIME_to_Calendar
<i2pr> [Slack/str4d] called from com.android.org.conscrypt.OpenSSLX509Certificate.getNotBefore()
<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?

Subtickets (add)

Change History (3)

comment:1 Changed 12 months ago by meeh

  • Milestone changed from 0.9.34 to 0.9.35

comment:2 Changed 12 months ago by meeh

  • Owner changed from str4d to meeh
  • Status changed from new to assigned

comment:3 Changed 6 months ago by zzz

  • Cc str4d added
  • Status changed from assigned to infoneeded

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

Note: See TracTickets for help on using tickets.