ehealthsolutions/element-x-ios
3
0
Fork 0
mirror of https://github.com/Novaloop-AG/element-x-ios.git synced 2026-11-03 16:10:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

Prevent crashes when computing aspect ratios on zero media width or height

Browse Source
This commit is contained in:
Stefan Ceriu
2024-02-08 17:17:46 +02:00
parent 1304f670c8
commit e1d1e99ad4
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ElementX/Sources/Services/Timeline/TimelineItems/RoomTimelineItemFactory.swift
+3 -3
View File
@@ -134,7 +134,7 @@ struct RoomTimelineItemFactory: RoomTimelineItemFactoryProtocol {
var aspectRatio: CGFloat? var aspectRatio: CGFloat?
let width = imageInfo.width.map(CGFloat.init) let width = imageInfo.width.map(CGFloat.init)
let height = imageInfo.height.map(CGFloat.init) let height = imageInfo.height.map(CGFloat.init)
if let width, let height { if let width, let height, width > 0, height > 0 {
aspectRatio = width / height aspectRatio = width / height
} }
@@ -488,7 +488,7 @@ struct RoomTimelineItemFactory: RoomTimelineItemFactoryProtocol {
let height = messageContent.info?.height.map(CGFloat.init) let height = messageContent.info?.height.map(CGFloat.init)
var aspectRatio: CGFloat? var aspectRatio: CGFloat?
if let width, let height { if let width, let height, width > 0, height > 0 {
aspectRatio = width / height aspectRatio = width / height
} }
@@ -508,7 +508,7 @@ struct RoomTimelineItemFactory: RoomTimelineItemFactoryProtocol {
let height = messageContent.info?.height.map(CGFloat.init) let height = messageContent.info?.height.map(CGFloat.init)
var aspectRatio: CGFloat? var aspectRatio: CGFloat?
if let width, let height { if let width, let height, width > 0, height > 0 {
aspectRatio = width / height aspectRatio = width / height
} }
changelog.d/pr-2437.bugfix
+1
View File
@@ -0,0 +1 @@
Prevent crashes when computing aspect ratios on zero media width or height