netty5

History

Shixiong Zhu 2457f386d8 Set `prev` to null when setting `cursor` to `head` in `scavengeSome`. Motivation: `scavengeSome()` has a corner case: when setting `cursor` to `head`, `this.prev` may point to the tail of the `WeakOrderQueue` linked list. Then it's possible that the following while loop will link the tail to the head, and cause endless loop. I made a reproducer in `36522e7b72` . The unit test will just run forever. Unfortunately, I cannot change it to a unit test because it needs to add some codes to `scavengeSome` to control the execution flow. Modification: Set `prev` to null when setting `cursor` to `head` in `scavengeSome` Result: Fixes #6153.	2017-01-07 20:48:46 +01:00
..
src	Set `prev` to null when setting `cursor` to `head` in `scavengeSome`.	2017-01-07 20:48:46 +01:00
pom.xml	Support compiling netty with Java9	2016-12-03 20:12:56 +01:00

Shixiong Zhu 2457f386d8 Set prev to null when setting cursor to head in scavengeSome.

Motivation:

`scavengeSome()` has a corner case: when setting `cursor` to `head`, `this.prev` may point to the tail of the  `WeakOrderQueue` linked list. Then it's possible that the following while loop will link the tail to the head, and cause endless loop.

I made a reproducer in 36522e7b72 . The unit test will just run forever. Unfortunately, I cannot change it to a unit test because it needs to add some codes to `scavengeSome` to control the execution flow.

Modification:

Set `prev` to null when setting `cursor` to `head` in `scavengeSome`

Result:

Fixes #6153.

2017-01-07 20:48:46 +01:00

src Set prev to null when setting cursor to head in scavengeSome. 2017-01-07 20:48:46 +01:00

pom.xml Support compiling netty with Java9 2016-12-03 20:12:56 +01:00