diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2024-11-06 10:25:21 +1100 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2024-11-07 12:46:44 +0100 |
commit | f6b546c6e4f036bc569df05cf76eced3f68d6db8 (patch) | |
tree | 200e4bc8e38b18ba3cc469e64f798006b14f721d | |
parent | 30b4f8816774665321e6903b4f55a929b015d16d (diff) | |
download | passt-f6b546c6e4f036bc569df05cf76eced3f68d6db8.tar passt-f6b546c6e4f036bc569df05cf76eced3f68d6db8.tar.gz passt-f6b546c6e4f036bc569df05cf76eced3f68d6db8.tar.bz2 passt-f6b546c6e4f036bc569df05cf76eced3f68d6db8.tar.lz passt-f6b546c6e4f036bc569df05cf76eced3f68d6db8.tar.xz passt-f6b546c6e4f036bc569df05cf76eced3f68d6db8.tar.zst passt-f6b546c6e4f036bc569df05cf76eced3f68d6db8.zip |
flow: Correct type of flowside_at_sidx()
Due to a copy-pasta error, this returns 'PIF_NONE' instead of NULL on the
failure case. PIF_NONE expands to 0, which turns into NULL, but it's
still confusing, so fix it. This removes a clang warning.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
-rw-r--r-- | flow_table.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/flow_table.h b/flow_table.h index a499e7b..f15db53 100644 --- a/flow_table.h +++ b/flow_table.h @@ -110,7 +110,7 @@ static inline const struct flowside *flowside_at_sidx(flow_sidx_t sidx) const union flow *flow = flow_at_sidx(sidx); if (!flow) - return PIF_NONE; + return NULL; return &flow->f.side[sidx.sidei]; } |