Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash after OCID DB is downloaded #888

Closed
3 tasks done
Osteri opened this issue Jul 3, 2016 · 1 comment · Fixed by #889
Closed
3 tasks done

Crash after OCID DB is downloaded #888

Osteri opened this issue Jul 3, 2016 · 1 comment · Fixed by #889

Comments

@Osteri
Copy link
Contributor

Osteri commented Jul 3, 2016

Thank you for your feedback! Please add below details so that we can help you a bit faster.


Agreements

Please check these boxes to ensure you've actually read and understood these important rules!


Overview

The generated getter function setRejCause expects a valid Integer object. The object should be coverted in to a primitive int, which does not happen, when the converted Integer value is null (and not a valid Integer): java.lang.NullPointerException: Attempt to invoke virtual method 'int java.lang.Integer.intValue()' on a null object reference

This is not a duplicate of https://github.com/CellularPrivacy/Android-IMSI-Catcher-Detector/issues/870, since it solves a different issue.

I have already made a fix for this and will make a pull request later. I created this issue just to trace this specific issue.

  • AIMSICD version (see the About screen): latest development branch; v0.1.43-alpha-SNAPSHOT-normal
  • Phone and ROM running current release: omnirom 6.0.1
  • Link to logfile without identifyable data:
06-02 01:42:49.770: E/AndroidRuntime(16818): FATAL EXCEPTION: main
06-02 01:42:49.770: E/AndroidRuntime(16818): Process: com.SecUpwN.AIMSICD, PID: 16818
06-02 01:42:49.770: E/AndroidRuntime(16818): java.lang.NullPointerException: Attempt to invoke virtual method 'int java.lang.Integer.intValue()' on a null object reference
06-02 01:42:49.770: E/AndroidRuntime(16818): at com.secupwn.aimsicd.utils.RealmHelper$3.execute(RealmHelper.java:453)

Reproduction

Clean installation; try to download OCID data, program forcefully closes (unhandled exception).


References

This problem is also referenced in @Nordlenning's comment: #870 (comment)

Note: there are 2 separate fatal exceptions at @Nordlenning's log output, this ticket is only created for the other exception.


Screenshots

@sciamano
Copy link

sciamano commented Jul 3, 2016

I see a crash too when trying to download the database. I did following: 1) clear all app data, 2) get ID-key, 3) start the DB download. At step 3 the app always crashes :(

Here is few lines from log-file, which I think are relevant

D/Injector(19087): Injection of com.secupwn.aimsicd.utils.RequestTask@6c3b9f took 6ms
D/roidIMSICatcherDetector(19087): BaseTask addTask activity:com.secupwn.aimsicd.ui.activities.MainActivity
I/RequestTask(19087): DBE_DOWNLOAD_REQUEST write to: /storage/emulated/0/Android/data/com.SecUpwN.AIMSICD/files/OpenCellID/opencell
id.csv
I/SignalStrengthTracker(19087): Ignored signal sample for CID: 5634267 due to device movement. Waiting for 20553 ms.
D/RequestTask(19087): doInBackground DBE_DOWNLOAD_REQUEST total not returned!
I/RealmHelper(19087): UpdateOpenCellID: OCID CSV size (lines): 621
D/RealmHelper(19087): PopulateDBeImport(): inserted 621 cells.
D/RealmHelper(19087): CheckDBe() Attempting to delete bad import data from Imports database...
D/AndroidRuntime(19087): Shutting down VM
I/Process (19087): Sending signal. PID: 19087 SIG: 9

after crash, the file /storage/emulated/0/Android/data/com.SecUpwN.AIMSICD/files/OpenCellID/opencell id.csv exists and contains some csv data

larsgrefer added a commit that referenced this issue Jul 6, 2016
Fix issue #888 NullPointerException by checking possible nulls.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants