Skip to content

Conversation

@ptgoetz
Copy link
Member

@ptgoetz ptgoetz commented Dec 13, 2013

Currently if random data is sent to Nimbus' thrift port, it will result in the following error, causing Nimbus to hang:

java.lang.OutOfMemoryError: Java heap space

Situations where this can happen:

  • Inadvertently ssh'ing to the nimbus port (ssh nimbus -p 6627)
  • Telnetting to the nimbus port (telnet nimbus 6627)
  • Network security/vulnerability scans that probe port 6627

The solution is to set a limit on the size of the buffers Thrift uses for reading messages. With this fix in place when any of the above occurs, the following will appear in the nimbus log (but nimbus will not hang or crash):

2013-12-13 17:10:52 o.a.t.s.TNonblockingServer [ERROR] Read a frame size of 1397966893, which is bigger than the maximum allowable buffer size for ALL connections.

@xumingming
Copy link
Contributor

+1

@asfgit asfgit merged commit 17de23c into apache:master Dec 19, 2013
asfgit pushed a commit that referenced this pull request Jul 16, 2014
update from apache/incubator-storm
asfgit pushed a commit that referenced this pull request Aug 25, 2014
Remove unncessary use of streams from GzipBridgeSerializationDelegate
asfgit pushed a commit that referenced this pull request Sep 22, 2014
jfota pushed a commit to mapr/incubator-storm that referenced this pull request Nov 5, 2014
HeartSaVioR pushed a commit to HeartSaVioR/storm that referenced this pull request Jan 20, 2015
asfgit pushed a commit that referenced this pull request Jan 22, 2015
Merge from apache/storm master to caofangkun/apache-storm master
knusbaum pushed a commit to knusbaum/incubator-storm that referenced this pull request Feb 11, 2015
Fixed merge conflicts with latest pull requests in community
knusbaum pushed a commit to knusbaum/incubator-storm that referenced this pull request Feb 11, 2015
Merge from storm/storm Master security
ptgoetz pushed a commit to ptgoetz/storm that referenced this pull request Aug 6, 2015
arunmahadevan pushed a commit to arunmahadevan/storm that referenced this pull request Aug 17, 2017
FIX some of the tests and metrics for the system bolt
ludovico99 added a commit to ludovico99/storm that referenced this pull request May 27, 2022
StefanoBelli added a commit to StefanoBelli/isw2-storm that referenced this pull request Mar 21, 2024
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.

3 participants