Sulkta-OSS/element-x-ada
3
0
Fork 0
Code Issues 1 Pull requests 10 Projects Releases Packages Wiki Activity Actions

Protect TimelineItemAspectRatioBox against Float.NaN (#1201)

Browse source 
* Protect `TimelineItemAspectRatioBox` against infinite values.
This commit is contained in:
Jorge Martin Espinosa 2023-08-31 12:58:37 +02:00 • committed by GitHub
parent e7cf36e2f9
commit 1c4eb45a60
2 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

1
changelog.d/1995.bugfix Normal file
View file

@ -0,0 +1 @@
Crash with `aspectRatio` modifier when `Float.NaN` was used as input.

4
features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/factories/event/TimelineItemContentMessageFactory.kt
View file

@ -132,10 +132,12 @@ class TimelineItemContentMessageFactory @Inject constructor(
} }
private fun aspectRatioOf(width: Long?, height: Long?): Float? { private fun aspectRatioOf(width: Long?, height: Long?): Float? {
return if (height != null && width != null) { val result = if (height != null && width != null) {
width.toFloat() / height.toFloat() width.toFloat() / height.toFloat()
} else { } else {
null null
} }
return result?.takeIf { it.isFinite() }
} }
} }