Skip to content

Commit 85b10f8

Browse files
committed
Fix version comparisons of different lengths
1 parent 22ad574 commit 85b10f8

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

SignalServiceKit/src/Util/AppVersion.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -196,8 +196,8 @@ public class AppVersion {
196196

197197
let largestCount = max(lhsComponents.count, rhsComponents.count)
198198
for index in (0..<largestCount) {
199-
let lhsComponent = parseVersionComponent(lhsComponents[index])
200-
let rhsComponent = parseVersionComponent(rhsComponents[index])
199+
let lhsComponent = parseVersionComponent(lhsComponents[safe: index])
200+
let rhsComponent = parseVersionComponent(rhsComponents[safe: index])
201201
if lhsComponent != rhsComponent {
202202
return (lhsComponent < rhsComponent) ? .orderedAscending : .orderedDescending
203203
}

SignalServiceKit/tests/Util/AppVersionTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import Foundation
77
import XCTest
88
@testable import SignalServiceKit
99

10-
final class AppVersionTests: XCTest {
10+
final class AppVersionTests: XCTestCase {
1111
func testVersionComparisons() {
1212
let testCases: [(String, String, ComparisonResult)] = [
1313
("0", "0", .orderedSame),

0 commit comments

Comments
 (0)