Skip to content

Commit

Permalink
Fix a bug in the TimeSyncController.js which causes time syncs to be …
Browse files Browse the repository at this point in the history
…blocked once the sync attempt failed
  • Loading branch information
dsilhavy committed Jun 18, 2021
1 parent 836395d commit 377428d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/dash/utils/TimelineConverter.js
Original file line number Diff line number Diff line change
Expand Up @@ -341,7 +341,7 @@ function TimelineConverter() {
}

function _onUpdateTimeSyncOffset(e) {
if (e.offset !== undefined) {
if (e.offset !== undefined && !isNaN(e.offset)) {
setClientTimeOffset(e.offset / 1000);
}
}
Expand Down
1 change: 1 addition & 0 deletions src/streaming/controllers/TimeSyncController.js
Original file line number Diff line number Diff line change
Expand Up @@ -547,6 +547,7 @@ function TimeSyncController() {

if (failed) {
lastTimingSource = null;
isSynchronizing = false;
errHandler.error(new DashJSError(Errors.TIME_SYNC_FAILED_ERROR_CODE, Errors.TIME_SYNC_FAILED_ERROR_MESSAGE));
}

Expand Down

0 comments on commit 377428d

Please sign in to comment.