fou: Prevent unbounded recursion in GUE error handler
authorStefano Brivio <sbrivio@redhat.com>
Mon, 17 Dec 2018 23:13:17 +0000 (00:13 +0100)
committerDavid S. Miller <davem@davemloft.net>
Tue, 18 Dec 2018 05:38:04 +0000 (21:38 -0800)
commit11789039da536fea96c98a40c2b441decf2e7323
tree68004178117f344d16b990edf36e7a18367aa24f
parenta28777f250311ac574237e6145ac72c7b3b7b049
fou: Prevent unbounded recursion in GUE error handler

Handling exceptions for direct UDP encapsulation in GUE (that is,
UDP-in-UDP) leads to unbounded recursion in the GUE exception handler,
syzbot reported.

While draft-ietf-intarea-gue-06 doesn't explicitly forbid direct
encapsulation of UDP in GUE, it probably doesn't make sense to set up GUE
this way, and it's currently not even possible to configure this.

Skip exception handling if the GUE proto/ctype field is set to the UDP
protocol number. Should we need to handle exceptions for UDP-in-GUE one
day, we might need to either explicitly set a bound for recursion, or
implement a special iterative handling for these cases.

Reported-and-tested-by: syzbot+43f6755d1c2e62743468@syzkaller.appspotmail.com
Fixes: b8a51b38e4d4 ("fou, fou6: ICMP error handlers for FoU and GUE")
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/fou.c