Bug 402032 - Exception in SMSPlugin onPacketReceived()
Summary: Exception in SMSPlugin onPacketReceived()
Status: RESOLVED FIXED
Alias: None
Product: kdeconnect
Classification: Applications
Component: android-application (show other bugs)
Version: 1.10
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Simon Redman
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-12-12 01:19 UTC by Keith Calvelli
Modified: 2018-12-12 01:26 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Relevant logcat (3.46 KB, text/plain)
2018-12-12 01:19 UTC, Keith Calvelli
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Keith Calvelli 2018-12-12 01:19:48 UTC
Created attachment 116869 [details]
Relevant logcat

SUMMARY
Request from Simon to create new bug ticket, from Bug 401677.  

STEPS TO REPRODUCE
1. From GSConnect, attempt to open Messaging window
2. Send a kdeconnect.sms.request_conversations packet to the device
3. Wait for a kdeconnect.sms.messages packet

OBSERVED RESULT
Window opens without SMS history from phone.  No kdeconnect.sms.messages packet sent to host.

EXPECTED RESULT
SMS history from phone should be shown in Messaging window.  

SOFTWARE/OS VERSIONS
Windows: 
MacOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Phone is Sony Xperia XA2 Ultra on Android 8.0

Logcat entries show an exception in org.kde.kdeconnect.Plugins.SMSPlugin.onPacketReceived() where the code calls Integer.parseInt() on a THREAD_ID of 9223372036854775807.

Note:  It looks like the code was updated on github 15 minutes ago.  I'll still go ahead and enter this bug as it was requested (not sure if you need for tracking purposes).
Comment 1 Simon Redman 2018-12-12 01:26:24 UTC
(In reply to Keith Calvelli from comment #0)

> Note:  It looks like the code was updated on github 15 minutes ago.  I'll
> still go ahead and enter this bug as it was requested (not sure if you need
> for tracking purposes).

Thanks! As you noticed, it is (hopefully) fixed. You should be able to download the latest from https://binary-factory.kde.org/view/Android/job/kdeconnect_android/lastSuccessfulBuild/artifact/kdeconnect-android/build/outputs/apk/debug/ if you don't want to wait for the next official release