crash on mms export

[farfromrefug]

I get a crash while trying to export my sms/mms. It happens at the end while exporting mms.

02-04 11:30:14.696 17391 17391 E AndroidRuntime: Process: com.github.tmo1.sms_ie, PID: 17391
02-04 11:30:14.696 17391 17391 E AndroidRuntime: java.lang.NullPointerException: address.getString(addressIndex) must not be null
02-04 11:30:14.696 17391 17391 E AndroidRuntime: 	at C0.f.b(SourceFile:41)
02-04 11:30:14.696 17391 17391 E AndroidRuntime: 	at H0.y.e(SourceFile:17)
02-04 11:30:14.696 17391 17391 E AndroidRuntime: 	at p1.b.c(SourceFile:9)
02-04 11:30:14.696 17391 17391 E AndroidRuntime: 	at D1.z.run(SourceFile:117)
02-04 11:30:14.696 17391 17391 E AndroidRuntime: 	at B.e.run(SourceFile:363)
02-04 11:30:14.696 17391 17391 E AndroidRuntime: 	at K1.j.run(SourceFile:3)
02-04 11:30:14.696 17391 17391 E AndroidRuntime: 	at K1.a.run(SourceFile:92)
02-04 11:30:14.696 17391 17391 E AndroidRuntime: 	Suppressed: I1.g: [c0{Cancelling}@df729ea, Dispatchers.Main]

My guess would be that it happens because of the fact that i actually first imported an old backup of mine and that i now am trying to export with newly added messages. Something must not be "filled" from the imported backup.

I cant create a sample to reproduce because i dont know which part of the data is corrupted i have more than 15000 messages :s
However the error seems pretty clear. I would gues the issue is here

sms-ie/app/src/main/java/com/github/tmo1/sms_ie/ImportExportMessages.kt

Line 218 in 96e3229

appContext, displayNames, address.getString(addressIndex)

sms-ie/app/src/main/java/com/github/tmo1/sms_ie/ImportExport.kt

Line 89 in 96e3229

fun lookupDisplayName(

lookupDisplayName expect address to be not null. I ll test and create a PR

[tmo1]

This is a variation of issue #38; the (contributed) fix for that issue (PR #39) added null checks to the callers of lookupDisplayName, but only to the SMS and call log export functions, and not to the MMS export function. Your PR seems to be a better solution: change the lookupDisplayName function signature to accept null values, and include a null check inside the function. I'll merge the PR, and then remove the now redundant null checks in the caller functions.

Thank you!