Message-ID: <gh-postgresql-interfaces-psqlodbc-18-rc1599798146@github.com>
From: "progmachine (@progmachine)" <noreply+progmachine@github.com>
To: "postgresql-interfaces/psqlodbc" <noreply+postgresql-interfaces-psqlodbc@github.com>
Date: Tue, 14 May 2024 10:44:08 +0000
Subject: Re: [postgresql-interfaces/psqlodbc] PR #18: Notes todo and attempt to fix memory leak
In-Reply-To: <gh-postgresql-interfaces-psqlodbc-18@github.com>
References: <gh-postgresql-interfaces-psqlodbc-18@github.com>
List-Id: <gh-postgresql-interfaces-psqlodbc.github.com>
X-GitHub-Author-Login: progmachine
X-GitHub-Comment-Id: 1599798146
X-GitHub-Comment-Type: review_comment
X-GitHub-Commit: f77bc41444c388207ffb7348b5e0c3259b09cc2e
X-GitHub-Issue: 18
X-GitHub-Line: 563
X-GitHub-Path: connection.c
X-GitHub-Repo: postgresql-interfaces/psqlodbc
X-GitHub-Type: review_comment
X-GitHub-Url: https://github.com/postgresql-interfaces/psqlodbc/pull/18#discussion_r1599798146
Content-Type: text/plain; charset=utf-8

(on connection.c:563)

You do not need to decrement `self->ntables` here, it will leave some entries out of this and next for-loop scopes.