Add more sanity checks around callers of changeDependencyFor()

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Add more sanity checks around callers of changeDependencyFor()
Date: 2023-06-28 23:36:30
Message-ID: ZJzD/rn+UbloKjB7@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi all,

While working on a different patch, I have noted three code paths that
call changeDependencyFor() but don't check that they do not return
errors. In all the three cases (support function, extension/schema
and object/schema), it seems to me that only one dependency update is
expected.

I am adding that to the next CF. Thoughts or comments about the
attached?
--
Michael

Attachment Content-Type Size
0001-Add-checks-for-values-returned-by-changeDependencyFo.patch text/x-diff 2.6 KB

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2023-06-28 23:42:09 Re: Changing types of block and chunk sizes in memory contexts
Previous Message Andres Freund 2023-06-28 23:34:09 Re: Changing types of block and chunk sizes in memory contexts