Re: [COMMITTERS] pgsql: Fix issues with pg_ctl

2014-03-05 Thread Stephen Frost
* Thom Brown (t...@linux.com) wrote:
> On 5 March 2014 06:33, Stephen Frost  wrote:
> > Fix issues with pg_ctl
[...]
> thom@swift ~/dbtest $ sh: 1: ���: not found

Yeesh.  Always with the simple ones it seems.  Yeah, I see what the
issue here is, will fix.

Thanks!

Stephen


signature.asc
Description: Digital signature


Re: [COMMITTERS] pgsql: Fix issues with pg_ctl

2014-03-05 Thread Thom Brown
On 5 March 2014 06:33, Stephen Frost  wrote:

> Fix issues with pg_ctl
>
> The new, small, free_readfile managed to have bug in it which could
> cause it to try and free something it shouldn't, and fix the case
> where it was being called with an invalid pointer leading to a
> segfault.
>
> Noted by Bruce, issues introduced and fixed by me.


Yeah, this has fixed the segfaults, thanks.

Now I get an issue when restarting my cluster, but not when I stop and
start:

thom@swift ~/dbtest $ pg_ctl stop -D primary
waiting for server to shut down done
server stopped
thom@swift ~/dbtest $ pg_ctl start -D primary
server starting
thom@swift ~/dbtest $ 2014-03-05 11:46:21.963 GMT -  - LOG:  redirecting
log output to logging collector process
2014-03-05 11:46:21.963 GMT -  - HINT:  Future log output will appear in
directory "pg_log".

thom@swift ~/dbtest $ pg_ctl restart -D primary
waiting for server to shut down done
server stopped
server starting
thom@swift ~/dbtest $ sh: 1: ���: not found

-- 
Thom


[COMMITTERS] pgsql: Fix issues with pg_ctl

2014-03-04 Thread Stephen Frost
Fix issues with pg_ctl

The new, small, free_readfile managed to have bug in it which could
cause it to try and free something it shouldn't, and fix the case
where it was being called with an invalid pointer leading to a
segfault.

Noted by Bruce, issues introduced and fixed by me.

Branch
--
master

Details
---
http://git.postgresql.org/pg/commitdiff/eb933162cdcbcaa5c56c75eb21b9c055af9748a0

Modified Files
--
src/bin/pg_ctl/pg_ctl.c |   12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)


-- 
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers