diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2003-03-31 20:32:50 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2003-03-31 20:32:50 +0000 |
| commit | 28295da0a52fa4ca1c9365d390f2e8265dedeebf (patch) | |
| tree | 0fc518dfabd5639aa877cf14dacd89745170f86c /src | |
| parent | ba4de4e06a28c1d3962cf9db34612aad504150ec (diff) | |
| download | postgresql-28295da0a52fa4ca1c9365d390f2e8265dedeebf.tar.gz postgresql-28295da0a52fa4ca1c9365d390f2e8265dedeebf.zip | |
TestConfiguration returns int, not bool. This mistake is relatively
harmless on signed-char machines but would lead to core dump in the
deadlock detection code if char is unsigned. Amazingly, this bug has
been here since 7.1 and yet wasn't reported till now. Thanks to Robert
Bruccoleri for providing the opportunity to track it down.
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/storage/lmgr/deadlock.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/backend/storage/lmgr/deadlock.c b/src/backend/storage/lmgr/deadlock.c index 8aeef9c36d6..4c25f2ce6df 100644 --- a/src/backend/storage/lmgr/deadlock.c +++ b/src/backend/storage/lmgr/deadlock.c @@ -12,7 +12,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/storage/lmgr/deadlock.c,v 1.15 2002/11/01 00:40:23 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/storage/lmgr/deadlock.c,v 1.15.2.1 2003/03/31 20:32:50 tgl Exp $ * * Interface: * @@ -47,7 +47,7 @@ typedef struct static bool DeadLockCheckRecurse(PGPROC *proc); -static bool TestConfiguration(PGPROC *startProc); +static int TestConfiguration(PGPROC *startProc); static bool FindLockCycle(PGPROC *checkProc, EDGE *softEdges, int *nSoftEdges); static bool FindLockCycleRecurse(PGPROC *checkProc, @@ -301,7 +301,7 @@ DeadLockCheckRecurse(PGPROC *proc) * number of soft edges. *-------------------- */ -static bool +static int TestConfiguration(PGPROC *startProc) { int softFound = 0; |