Fix boolean assignment of return values to use "FALSE" rather than the

 mis-copied "NULL", which happens to have the same binary value.
Previously, gcc gave non-fatal warnings.

diff --git a/src/backend/utils/adt/network.c b/src/backend/utils/adt/network.c
index ac40afeef6a7e4c1365b60c11bb2f11339d51ae6..17caa23cb4923e4115d85f66c21096fdf6e003be 100644 (file)
--- a/src/backend/utils/adt/network.c
+++ b/src/backend/utils/adt/network.c
@@ -221,7 +221,7 @@ bool
 network_sub(inet *a1, inet *a2)
 {
        if (!PointerIsValid(a1) || !PointerIsValid(a2))
-               return NULL;
+               return FALSE;
                
        if ((ip_family(a1) == AF_INET) && (ip_family(a2) == AF_INET))
        {
@@ -241,7 +241,7 @@ bool
 network_subeq(inet *a1, inet *a2)
 {
        if (!PointerIsValid(a1) || !PointerIsValid(a2))
-               return NULL;
+               return FALSE;
 
        if ((ip_family(a1) == AF_INET) && (ip_family(a2) == AF_INET))
        {
@@ -261,7 +261,7 @@ bool
 network_sup(inet *a1, inet *a2)
 {
        if (!PointerIsValid(a1) || !PointerIsValid(a2))
-               return NULL;
+               return FALSE;
 
        if ((ip_family(a1) == AF_INET) && (ip_family(a2) == AF_INET))
        {
@@ -281,7 +281,7 @@ bool
 network_supeq(inet *a1, inet *a2)
 {
        if (!PointerIsValid(a1) || !PointerIsValid(a2))
-               return NULL;
+               return FALSE;
 
        if ((ip_family(a1) == AF_INET) && (ip_family(a2) == AF_INET))
        {