Skip to content

Commit 69979aa

Browse files
uercegnonelse
authored andcommitted
fix: guard against dispatching nil semaphore during e2e tests
1 parent 0a91e6e commit 69979aa

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

Adjust/Internal/ADJPackageHandler.m

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -328,11 +328,19 @@ - (void)sendNextI:(ADJPackageHandler *)selfI
328328
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(waitTime * NSEC_PER_SEC)),
329329
self.internalQueue, ^{
330330
[self.logger verbose:@"Package handler finished waiting to continue"];
331-
dispatch_semaphore_signal(self.sendingSemaphore);
331+
if (selfI.sendingSemaphore == nil) {
332+
[self.logger error:@"Sending semaphore is nil"];
333+
return;
334+
}
335+
dispatch_semaphore_signal(selfI.sendingSemaphore);
332336
[self sendFirstPackage];
333337
});
334338
} else {
335339
// otherwise just signal and send next
340+
if (selfI.sendingSemaphore == nil) {
341+
[self.logger error:@"Sending semaphore is nil"];
342+
return;
343+
}
336344
dispatch_semaphore_signal(selfI.sendingSemaphore);
337345
[selfI sendFirstI:selfI];
338346
}

0 commit comments

Comments
 (0)