Skip to content

Hardenize libibverbs initialisation

Yann Droneaud linux-rdma 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 3 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5load_drivers: reset the list head after releasing itemssrc/init.c+2 -00 replies
2/5ibverbs_init: reset the list head after releasing itemssrc/init.c+2 -00 replies
3/5read_config: don't try to open file beginning with '.'src/init.c+3 -02 replies
4/5read_config: ignore directory entry with backup suffix (~)src/init.c+3 -00 replies
5/5read_config_file: ignore driver line without driver namesrc/init.c+1 -11 reply

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/5] Hardenize libibverbs initialisation

In order to hardenize my test application, I tried "fuzzing" its input with zzuf[1]. But my application mostly failed to initialize due to unhandled failure cases.

So here're are patches to clean the situation and let's application exit properly.

The patches here are against kernel.org's libibverbs repository[2]

[1] http://caca.zoy.org/wiki/zzuf
[2] http://git.kernel.org/?p=libs/infiniband/libibverbs.git

Yann Droneaud (5):
load_drivers: reset the list head after releasing items
ibverbs_init: reset the list head after releasing items
read_config: don't try to open file beginning with '.'
read_config: ignore directory entry with backup suffix (~)
read_config_file: ignore driver line without driver name

 src/init.c |   12 +++++++++++-
 1 files changed, 11 insertions(+), 1 deletions(-)
Signature · 6 lines
1.7.4.2

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html