On 2026-03-14 Sa 11:48 PM, Mahendra Singh Thalor wrote:
> Hi hackers,
>
> +++ b/src/bin/pg_dump/pg_restore.c
> @@ -1268,16 +1268,6 @@ restore_all_databases(const char *inputFileSpec,
> */
> memcpy(tmpopts, original_opts, sizeof(RestoreOptions));
>
> - /*
> - * We need to reset override_dbname so that objects
> can be restored
> - * into an already created database. (used with
> -d/--dbname option)
> - */
> - if (tmpopts->cparams.override_dbname)
> - {
> - pfree(tmpopts->cparams.override_dbname);
> - tmpopts->cparams.override_dbname = NULL;
> - }
> -
>
> In commit 77152d26efefde2eed4, we added some functions and we missed
> to remove some unnecessary lines. Those lines are not needed as we are
> using memcpy to get the original copy before these lines.
>
> Here, I am attaching a patch to fix this. Please review it.
Thanks. Pushed.
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
