<html dir="ltr"><head></head><body style="text-align:left; direction:ltr;" bgcolor="#fcfcfc" text="#232627" link="#2980b9" vlink="#232627"><div>On Tue, 2022-11-22 at 10:30 -0500, okaya@kernel.org wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><pre>From: Sinan Kaya <</pre><a href="mailto:okaya@kernel.org"><pre>okaya@kernel.org</pre></a><pre>></pre><pre><br></pre><pre>Following up the codeql reported problems first submitted</pre><pre>by Stephen Hemminger here:</pre><pre><br></pre><a href="https://lore.kernel.org/all/20220527161210.77212d0b@hermes.local/t/"><pre>https://lore.kernel.org/all/20220527161210.77212d0b@hermes.local/t/</pre></a><pre><br></pre><pre><br></pre><pre>Posting a series of fixes about potential null pointer accesses.</pre><pre><br></pre><pre>Changes from v1:</pre><pre>- Remove braces around single line statements</pre><pre>- use NULL comparisons</pre><pre><br></pre><pre>Sinan Kaya (11):</pre><pre>  ethdev: check return result of rte_eth_dev_info_get</pre><pre>  net/tap: check if name is null</pre><pre>  memzone: check result of rte_fbarray_get</pre><pre>  memzone: check result of malloc_elem_from_data</pre><pre>  malloc: malloc_elem_join_adjacent_free can return null</pre><pre>  malloc: check result of rte_mem_virt2memseg_list</pre><pre>  malloc: check result of rte_fbarray_get</pre><pre>  malloc: check result of rte_mem_virt2memseg</pre><pre>  malloc: check result of malloc_elem_free</pre><pre>  malloc: check result of elem_start_pt</pre><pre>  bus/vdev: check result of rte_vdev_device_name</pre><pre><br></pre><pre> drivers/net/tap/rte_eth_tap.c        |  3 +++</pre><pre> lib/eal/common/eal_common_memalloc.c |  5 ++++-</pre><pre> lib/eal/common/eal_common_memzone.c  | 10 +++++++++-</pre><pre> lib/eal/common/malloc_elem.c         | 14 +++++++++++---</pre><pre> lib/eal/common/malloc_heap.c         |  9 ++++++++-</pre><pre> lib/ethdev/ethdev_vdev.h             |  2 ++</pre><pre> lib/ethdev/rte_class_eth.c           |  4 +++-</pre><pre> 7 files changed, 40 insertions(+), 7 deletions(-)</pre><pre><br></pre><pre><br></pre></blockquote><pre><br></pre><pre><br></pre><pre>Checking to see if I need to cover any feedback.</pre><pre><br></pre><pre><br></pre></body></html>