Project

General

Profile

Bug #384

NullPointerException dans MemorizingTrustManager

Added by Romain Vimont almost 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
Start date:
01/07/2012
Due date:
% Done:

100%

Estimated time:
Affected version:
0.1.6
Locale:
Close

Description

En me connectant sur un serveur qui me présentait un certificat autosigné, quand je l'ai accepté, j'ai eu un NullPointerException ligne 384 de MemorizingTrustManager, dans ce bout de code :

    public static void interactResult(Intent i) {
        int decisionId = i.getIntExtra(DECISION_INTENT_ID, MTMDecision.DECISION_INVALID);
        int choice = i.getIntExtra(DECISION_INTENT_CHOICE, MTMDecision.DECISION_INVALID);
        Log.d(TAG, "interactResult: " + decisionId + " chose " + choice);
        Log.d(TAG, "openDecisions: " + openDecisions);

        MTMDecision d;
        synchronized(openDecisions) {
             d = openDecisions.get(decisionId);
             openDecisions.remove(decisionId);
        }
        synchronized(d) {             // <--- ligne 384 : d is Null
            d.state = choice;
            d.notify();
        }
    }

Je ne sais pas trop la cause (est-ce qu'un simple test d != null suffirait, où alors cela traduit-il un problème plus important, où d ne devrait jamais être null ?).


Related issues

Related to Feature #408: Update MITM library to the latest versionClosed2012-01-31

Associated revisions

Revision 975:d8305c375b10 (diff)
Added by Frédéric Barthéléry over 6 years ago

Update to the latest version of MemorizingTrustManager
Also fix the LeakedReceiver issue.
Fix bug #384
close #408
Mitm commit: 11b37471c5403c0510619278d453ca393405f162

Revision 998:d8305c375b10 (diff)
Added by Frédéric Barthéléry over 6 years ago

Update to the latest version of MemorizingTrustManager
Also fix the LeakedReceiver issue.
Fix bug #384
close #408
Mitm commit: 11b37471c5403c0510619278d453ca393405f162

History

#1 Updated by Frédéric Barthéléry almost 7 years ago

  • Status changed from New to Assigned
  • Assignee set to Frédéric Barthéléry

Salut Rom1,

Peux tu me poster les logs de Logcat ? C'est effectivement un probleme plus grave, d ne devrait jamais etre null

#2 Updated by Frédéric Barthéléry almost 7 years ago

  • Target version set to 0.1.8

#3 Updated by Frédéric Barthéléry over 6 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 0 to 100

#4 Updated by Frédéric Barthéléry over 6 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF