Skip to content

Fixes for coverity issues in fbarray

Anatoly Burakov dpdk-dev 1 version Applied

Status from this message: Applied, thanks

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 2 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/4fbarray: use strlcpy instead of snprintflib/librte_eal/common/eal_common_fbarray.c+1 -10 replies
2/4fbarray: add check for failed file descriptor openlib/librte_eal/common/eal_common_fbarray.c+5 -00 replies
3/4fbarray: fix potential null-dereferencelib/librte_eal/common/eal_common_fbarray.c+2 -10 replies
4/4fbarray: handle negative returnlib/librte_eal/common/eal_common_fbarray.c+3 -00 replies

Reviews trailers per patch

None of the 4 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchAdrien Mazarguil
SeriesA-b
1/4
2/4
3/4
4/4

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/4] Fixes for coverity issues in fbarray

This patchset fixes a number of Coverity issues introduced in recent DPDK memory hotplug patchset.

Coverity issues fixed:
- 272564 - error condition not checked
- 272579 - dereference before null check
- 272586 - error condition not checked

There are two additional issues reported by coverity:
- 272598 - error condition not checked
- 272599 - error condition not checked

However, they are fixed by a separate patch [1].

[1] http://dpdk.org/dev/patchwork/patch/38091/

Anatoly Burakov (4):
fbarray: use strlcpy instead of snprintf
fbarray: add check for failed file descriptor open
fbarray: fix potential null-dereference
fbarray: handle negative return

 lib/librte_eal/common/eal_common_fbarray.c | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)
Signature · 1 line
2.7.4