In
c8ad4d8166a dlist_member_check()'s arguments were made const. Unfortunately
the implementation of dlist_member_check() used dlist_foreach(), which
currently doesn't work for const lists.
As a workaround, open-code the list iteration. The other check functions
already do so.
Discussion: https://postgr.es/m/
20230118182214.co7dp4oahiunwg57@awork3.anarazel.de
index f06febd6985781eb1dcf2c546d52e7cc10da8b37..aeb7f75cd0c3b49a082b640db1bfff4cb909f667 100644 (file)
@@ -59,11 +59,12 @@ slist_delete(slist_head *head, const slist_node *node)
void
dlist_member_check(const dlist_head *head, const dlist_node *node)
{
- dlist_iter iter;
+ const dlist_node *cur;
- dlist_foreach(iter, head)
+ /* iteration open-coded to due to the use of const */
+ for (cur = head->head.next; cur != &head->head; cur = cur->next)
{
- if (iter.cur == node)
+ if (cur == node)
return;
}
elog(ERROR, "double linked list member check failure");