pgAdmin 4 commit: Fix usage of QString::toUtf8(). Fixes #2299 - Mailing list pgadmin-hackers

From Dave Page
Subject pgAdmin 4 commit: Fix usage of QString::toUtf8(). Fixes #2299
Date
Msg-id E1csqCa-0000r7-VH@gothos.postgresql.org
Whole thread Raw
List pgadmin-hackers
Fix usage of QString::toUtf8(). Fixes #2299

Desktop runtime has a couple bugs using already freed memory supplied to embedded python leading to occasional crashes
andfailures to start maintenance processes (VACUUM, ANALYZE, REINDEX, CLUSTER). 

This is caused by incorrect usage of QString::toUtf8() which returns newly created QByteArray which is automatically
destroyedonce closing context; e.g. 
Py_SetPythonHome(pythonHome.toUtf8().data());

Branch
------
master

Details
-------
http://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=72548a99e142ec508e815a7e50c5611e2fe34407
Author: Maxim Zakharov <dp.maxime@gmail.com>

Modified Files
--------------
runtime/Server.cpp | 32 ++++++++++++++++++++------------
runtime/Server.h   |  5 +++++
2 files changed, 25 insertions(+), 12 deletions(-)


pgadmin-hackers by date:

Previous
From: Khushboo Vashi
Date:
Subject: Re: [pgAdmin4][Patch]: Fixed Ascending/Descendingorder in backgrid
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Add header that seems to be needed with Python 3.