Hi On Tue, Dec 15, 2020 at 12:37 PM Gert Doering <g...@greenie.muc.de> wrote:
> Hi, > > On Tue, Dec 15, 2020 at 06:16:00PM +0100, Domagoj Pensa wrote: > > When registering DNS on Windows, argv is freed after being used in first > > ipconfig command (/flushdns). > > > > Then same argv is used uninitialized in next ipconfig command > (/registerdns) > > causing heap exception and subprocess crash. > > > > As a consequence second command is never executed and locked netcmd > > semaphore is not cleanly released. > > > > Removing argv freeing between ipconfig calls solves the problem. > > Oh! Yes, now with your patch, this is very obvious - there is a trac > ticket (so when I merge this, I'll add the trac ticket number to the > commit message) but it sort of puzzled Selva and me, because the > code "looked sane". > Indeed, I went looking at wrong places. Thanks, Selva
_______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
