Skip to content

INT-4366: Fix MulticastSendingMessageHandler - #2329

Merged
garyrussell merged 2 commits into
spring-projects:masterfrom
artembilan:INT-4366
Jan 19, 2018
Merged

garyrussell merged 2 commits into
spring-projects:masterfrom
artembilan:INT-4366

Conversation

@artembilan

Copy link
Copy Markdown
Member

JIRA: https://jira.spring.io/browse/INT-4366

Fix race condition in the MulticastSendingMessageHandler around
multicastSocket and super socket properties.

  • Synchronize around this and check for the multicastSocket == null.
    This let the MulticastSendingMessageHandler to fully configure and
    prepare the socket for use.
  • Remove socket.setInterface(whichNic) since it is populated by the
    InetSocketAddress ctor before

Cherry-pick to 4.3.x

JIRA: https://jira.spring.io/browse/INT-4366

Fix race condition in the `MulticastSendingMessageHandler` around
`multicastSocket` and super `socket` properties.

* Synchronize around `this` and check for the `multicastSocket == null`.
This let the `MulticastSendingMessageHandler` to fully configure and
prepare the socket for use.
* Remove `socket.setInterface(whichNic)` since it is populated by the
`InetSocketAddress` ctor before

**Cherry-pick to 4.3.x**
@garyrussell
garyrussell merged commit c3b64dc into spring-projects:master Jan 19, 2018
@garyrussell

Copy link
Copy Markdown
Contributor

Unfortunately, there are many conflicts when cherry-picking - perhaps I should just backport the JIRA fix?

@artembilan

Copy link
Copy Markdown
Member Author

perhaps I should just backport the JIRA fix?

Fine with me.
Thanks

@artembilan
artembilan deleted the INT-4366 branch January 22, 2018 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants