aboutgitcodebugslistschat
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2024-11-06 10:25:21 +1100
committerStefano Brivio <sbrivio@redhat.com>2024-11-07 12:46:44 +0100
commitf6b546c6e4f036bc569df05cf76eced3f68d6db8 (patch)
tree200e4bc8e38b18ba3cc469e64f798006b14f721d
parent30b4f8816774665321e6903b4f55a929b015d16d (diff)
downloadpasst-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.h2
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];
}