diff options
| author | Ben Pfaff <blp@ovn.org> | 2021-05-06 10:53:17 -0700 |
|---|---|---|
| committer | Ilya Maximets <i.maximets@ovn.org> | 2022-05-25 21:22:15 +0200 |
| commit | 91d3fc87ac96ee26f8d60e3adb25ffa975c12af8 (patch) | |
| tree | a2746c51d1a4dca68e29e1f29e0043b0d1b10811 | |
| parent | 78a5ad8051a6339870c6b240ae73fe48336d9776 (diff) | |
| download | openvswitch-91d3fc87ac96ee26f8d60e3adb25ffa975c12af8.tar.gz | |
classifier: Make find_match_wc() prototype and definition match.
The prototype said *, the definition said [CLS_MAX_TRIES]. GCC 11
complains about this (though it is perfectly valid from a C standards
perspective). It would probably be better to make them both use
[CLS_MAX_TRIES] but that's only allowed if the struct's definition is
visible at the point of the prototype, which it's not. Instead of
moving the definition, this commit just changes both usages to *.
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Paolo Valerio <pvalerio@redhat.com>
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
| -rw-r--r-- | lib/classifier.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/classifier.c b/lib/classifier.c index 2a1d155da..c4790ee6b 100644 --- a/lib/classifier.c +++ b/lib/classifier.c @@ -1683,7 +1683,7 @@ find_match(const struct cls_subtable *subtable, ovs_version_t version, static const struct cls_match * find_match_wc(const struct cls_subtable *subtable, ovs_version_t version, - const struct flow *flow, struct trie_ctx trie_ctx[CLS_MAX_TRIES], + const struct flow *flow, struct trie_ctx *trie_ctx, unsigned int n_tries, struct flow_wildcards *wc) { if (OVS_UNLIKELY(!wc)) { |