zebra: fix wrong hash count function call - #21512
Conversation
Greptile SummaryThis PR fixes a merge conflict between #21263 (which added Confidence Score: 5/5Safe to merge β targeted, correct bug fix with no side effects. Single-file change that replaces No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[zebra_l3vni struct] --> B["rmac_table\n(struct hash *)"]
A --> C["nh_table\n(zebra_neigh_db_head β typesafe)"]
B --> D["hashcount(rmac_table)\nβ
correct for struct hash *"]
C --> E["hashcount(nh_table)\nβ BEFORE β wrong type"]
C --> F["zebra_neigh_db_count(nh_table)\nβ
AFTER β correct for typesafe hash"]
style E fill:#ffcccc,stroke:#cc0000
style F fill:#ccffcc,stroke:#00cc00
style D fill:#ccffcc,stroke:#00cc00
Reviews (1): Last reviewed commit: "zebra: fix wrong hash count function cal..." | Re-trigger Greptile |
ae9e9db to
425e49f
Compare
The Pull Requests FRRouting#21263 and FRRouting#21388 had conflicting changes: - 21263 uses `hashcount` (lib/hash.h implementation) to count items in a hash - 21388 changes the hash type being referenced in the other PR with type safe hash (lib/typesafe.h) Since both Pull Requests got merged at nearly the same time the CI didn't have a chance to build them together. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
425e49f to
2701a9a
Compare
The Pull Requests #21263 and #21388 had conflicting changes:
hashcount(lib/hash.h implementation) to count items in a hashSince both Pull Requests got merged at nearly the same time the CI didn't have a chance to build them together.