From 614253fa24e294ea6a9bac8b601c70ba41d1fb10 Mon Sep 17 00:00:00 2001 From: heck Date: Tue, 3 Mar 2020 01:23:00 +0100 Subject: [PATCH] Added test case for JNI-88 --- .hgignore | 3 + test/JNI_88/JNI_88.java | 119 ++++++++++++++++++++++++++++++++++++++++ test/JNI_88/Makefile | 26 +++++++++ 3 files changed, 148 insertions(+) create mode 100644 test/JNI_88/JNI_88.java create mode 100644 test/JNI_88/Makefile diff --git a/.hgignore b/.hgignore index 919e7bd..e3351a7 100644 --- a/.hgignore +++ b/.hgignore @@ -85,3 +85,6 @@ android/external/[^.]* local.conf src/local.conf +# Default ignored files +.idea/workspace.xml +.idea/vcs.xml diff --git a/test/JNI_88/JNI_88.java b/test/JNI_88/JNI_88.java new file mode 100644 index 0000000..e0e79a5 --- /dev/null +++ b/test/JNI_88/JNI_88.java @@ -0,0 +1,119 @@ +import foundation.pEp.jniadapter.*; +import java.util.Vector; +import java.net.URL; +import java.net.URLClassLoader; +import java.lang.Thread; +import java.lang.InterruptedException; + +import java.io.IOException; +import java.nio.file.Files; +import java.nio.file.Path; +import java.nio.file.Paths; + + +/* +This test is trying to reproduce the problem described in JNI-88 +https://pep.foundation/jira/browse/JNI-88 + +During the start up of the (banking-)connector we can see this behavior: +``` +java: throw_pEp_exception.cc:268: jint pEp::JNIAdapter::throw_pEp_Exception(JNIEnv*, PEP_STATUS): Assertion `ex' failed. + +Aborted (core dumped) +``` +We try to set OwnId simultaneously from two threads(with two different engine instances). There is also high load on the system (pEp-unreleted). + +Tuning tracing on in pgp_sequoia.c reveals this: +``` +15:02 < jonas> cert_find: (339945131124ECCBD16D46104F470C66E44B21E4, 0) +15:02 < jonas> key_load: -> PEP_KEY_NOT_FOUND +15:02 < jonas> cert_find: Looking up 339945131124ECCBD16D46104F470C66E44B21E4: PEP_KEY_NOT_FOUND +15:02 < jonas> cert_find: (339945131124ECCBD16D46104F470C66E44B21E4, 0) -> PEP_KEY_NOT_FOUND +15:02 < jonas> cert_save: Saving certificate: database is locked: PEP_UNKNOWN_ERROR +15:02 < jonas> cert_save: (339945131124ECCBD16D46104F470C66E44B21E4) -> PEP_UNKNOWN_ERROR +15:02 < jonas> pgp_generate_keypair: saving TSK: PEP_CANNOT_CREATE_KEY +15:02 < jonas> pgp_generate_keypair: -> PEP_CANNOT_CREATE_KEY +15:02 < jonas> cert_save: (1863F66D52A7E79B69DBFCD84EB613332303DE33) -> PEP_STATUS_OK + +[11:14] < dietz> | Schreib' Dir einfach was, das aus ein paar threads zyklisch myself() ruft (für jeweils neue Identitäten). +[11:14] < dietz> | Sollte nach wenigen Sekunden krachen. +[11:15] < dietz> | Wobei fdik in #dev meinte, wir dürfen myself() nicht parallel rufen. Scherzkeks. +[11:15] < dietz> | Spätestens beim plugin *müssen* wir aus zwei Prozessen auf die DB lesen und schreiben. +*/ + +class TestThread extends Thread { + private String threadName; + private int numIter = 1000000000; + private Engine e; + private Identity i1; + + TestThread( String name) { + threadName = name; + try { + e = new Engine(); + i1 = new Identity(); + i1.user_id = "pEp_own_userId"; + i1.me = true; + i1.username = "tTest User"; + i1.address = "hulu@hu.hu"; + } + catch (pEpException ex) { + System.out.println(threadName + ": cannot load"); + System.exit(0); + return; + } + System.out.println(threadName + ": Engine loaded"); + } + + private void testMyself() { + System.out.print("testMyself: -> "); + i1 = e.myself(i1); + System.out.println(i1.fpr); + } + + private void testMyselfWithName(String name) { + System.out.print("testMyselfWithName: " + name + " -> "); + Identity i2 = new Identity(); + i2.me = true; + i2.user_id = name; + i2.username = name; + i2.address = name + "@test.org"; + i2 = e.myself(i2); + System.out.println("FPR: " + i2.fpr); + } + + public void run() { + try { + for(int i = 0; i < numIter; i++) { +// testMyself(); + testMyselfWithName(threadName + Integer.toString(i)); +// Thread.sleep(0); + } + } catch (Exception e) { + System.out.println("Exception in Thread " + threadName); + System.out.println(e.toString()); + System.exit(0); + } + } +} + +class JNI_88 { + public static void main(String[] args) { + int numThreads = 2; + Vector vecT = new Vector(); + System.out.println("Creating num threads: " + numThreads); + + for(int i=0; i < numThreads; i++) { + String tName = "Thread-" + String.valueOf(i); + System.out.println("Creating Thread: " + tName); + vecT.add(new TestThread(tName)); + } + + System.out.println("num threads created: " + vecT.size()); + + for(Thread t : vecT) { + System.out.println("Starting Thread: " + t.getName()); + t.start(); + } + } +} diff --git a/test/JNI_88/Makefile b/test/JNI_88/Makefile new file mode 100644 index 0000000..179ad0e --- /dev/null +++ b/test/JNI_88/Makefile @@ -0,0 +1,26 @@ +include ../../Makefile.conf +include ../Makefile.conf + +JAVA_CLASSES = \ + JNI_88.class + +.PHONY: compile run test clean + +all: compile + $(MAKE) run + +run: compile + HOME=$(PWD) CLASSPATH=$(CLASSPATH) time $(VM) JNI_88 + +compile: $(JAVA_CLASSES) + +%.class: %.java + CLASSPATH=$(CLASSPATH) javac $< + +clean: + rm -f *.class + rm -f *.log + rm -rf .pEp_* + rm -Rf .pEp + rm -Rf .gnupg + rm -Rf .lldb