Skip to content

Commit 7679666

Browse files
authored
Merge pull request #1 from Horcrux7/agent-failed
Agent should failed on SocketNotFoundException jitsi#280
2 parents a53b402 + d9245d9 commit 7679666

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/main/java/org/ice4j/ice/ConnectivityCheckClient.java

+4-3
Original file line numberDiff line numberDiff line change
@@ -344,15 +344,15 @@ protected TransactionID startCheckForPair(
344344
{
345345
logger.trace("checking pair " + candidatePair + " tid " + tran);
346346
}
347+
348+
return tran;
347349
}
348350
catch (NetAccessManager.SocketNotFoundException e)
349351
{
350352
logger.info("Could not start connectivity check: " + e.getMessage());
351353
}
352354
catch (Exception ex)
353355
{
354-
tran = null;
355-
356356
IceSocketWrapper stunSocket = localCandidate.getStunSocket(null);
357357

358358
if (stunSocket != null)
@@ -376,7 +376,7 @@ protected TransactionID startCheckForPair(
376376
}
377377
}
378378

379-
return tran;
379+
return null;
380380
}
381381

382382
/**
@@ -944,6 +944,7 @@ protected void run()
944944
"Pair failed: "
945945
+ pairToCheck.toShortString());
946946
pairToCheck.setStateFailed();
947+
updateCheckListAndTimerStates(pairToCheck);
947948
}
948949
else
949950
{

0 commit comments

Comments
 (0)