Page MenuHomeFreeBSD
Differential D10922

limit buffer sizes.
ClosedPublic
Actions

Authored by delphij on May 26 2017, 7:44 AM.
Tags
None
Referenced Files
F173585667: D10922.diff
Sun, Sep 27, 12:22 AM
F173576609: D10922.id28893.diff
Sat, Sep 26, 11:02 PM
Unknown Object (File)
Sat, Sep 26, 1:11 AM
Unknown Object (File)
Sat, Sep 26, 1:07 AM
Unknown Object (File)
Fri, Sep 25, 8:41 PM
Unknown Object (File)
Fri, Sep 25, 8:38 PM
Unknown Object (File)
Fri, Sep 25, 3:01 PM
Unknown Object (File)
Tue, Sep 22, 7:59 PM
View All Files
Subscribers
imp

Details

Reviewers
cem
des
emaste
rmacklem
alfred
Commits
rS319369: * limit size of buffers to RPC_MAXDATASIZE
Summary
  • limit size of buffers to RPC_MAXDATASIZE
  • don't leak memory
  • be more picky about bad parameters

From:

https://raw.githubusercontent.com/guidovranken/rpcbomb/master/libtirpc_patch.txt
https://github.com/guidovranken/rpcbomb/blob/master/rpcbind_patch.txt

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

delphij created this revision.May 26 2017, 7:44 AM
Herald added a subscriber: imp. · View Herald TranscriptMay 26 2017, 7:44 AM
Harbormaster completed remote builds in B9499: Diff 28839.May 26 2017, 7:44 AM
cem accepted this revision.May 26 2017, 2:22 PM
This revision is now accepted and ready to land.May 26 2017, 2:22 PM
emaste accepted this revision.May 26 2017, 2:57 PM

LGTM

lib/libc/rpc/rpc_generic.c
612–614 ↗(On Diff #28839)

style(9) for consistency?

delphij updated this revision to Diff 28893.May 27 2017, 1:29 AM
delphij edited edge metadata.

style(9) fixes for new code.

This revision now requires review to proceed.May 27 2017, 1:29 AM
Harbormaster completed remote builds in B9519: Diff 28893.May 27 2017, 1:29 AM
delphij added a reviewer: alfred.May 27 2017, 1:29 AM
emaste accepted this revision.May 27 2017, 1:53 AM
This revision is now accepted and ready to land.May 27 2017, 1:53 AM
Closed by commit rS319369: * limit size of buffers to RPC_MAXDATASIZE (authored by delphij). · Explain WhyJun 1 2017, 6:12 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
lib/
libc/
rpc/
8 lines
17 lines
9 lines
xdr/
30 lines
sys/
rpc/
8 lines
5 lines
15 lines
xdr/
30 lines
usr.sbin/
rpcbind/
4 lines

Diff 29108

View Options

head/lib/libc/rpc/rpc_generic.c

Loading...
View Options

head/lib/libc/rpc/rpcb_prot.c

Loading...
View Options

head/lib/libc/rpc/rpcb_st_xdr.c

Loading...
View Options

head/lib/libc/xdr/xdr.c

Loading...
View Options

head/sys/rpc/rpc_generic.c

Loading...
View Options

head/sys/rpc/rpcb_clnt.c

Loading...
View Options

head/sys/rpc/rpcb_prot.c

Loading...
View Options

head/sys/xdr/xdr.c

Loading...
View Options

head/usr.sbin/rpcbind/rpcb_svc_com.c

Loading...