From: Alvaro Herrera <alvherre@alvh.no-ip.org>
Date: Tue, 23 Jul 2013 21:23:28 +0000 (-0400)
Subject: Check for NULL result from strdup
X-Git-Url: http://git.postgresql.org/gitweb/?a=commitdiff_plain;h=bb686c9a865dc15a704e6a96367b3d7bfe79df6f;p=users%2Fbernd%2Fpostgres.git

Check for NULL result from strdup

Per Coverity Scan
---

diff --git a/src/interfaces/libpq/fe-secure.c b/src/interfaces/libpq/fe-secure.c
index 174cf426f0..a6a09cd1ab 100644
--- a/src/interfaces/libpq/fe-secure.c
+++ b/src/interfaces/libpq/fe-secure.c
@@ -1131,7 +1131,17 @@ initialize_SSL(PGconn *conn)
 		{
 			/* Colon, but not in second character, treat as engine:key */
 			char	   *engine_str = strdup(conn->sslkey);
-			char	   *engine_colon = strchr(engine_str, ':');
+			char	   *engine_colon;
+
+			if (engine_str == NULL)
+			{
+				printfPQExpBuffer(&conn->errorMessage,
+								  libpq_gettext("out of memory\n"));
+				return -1;
+			}
+
+			/* cannot return NULL because we already checked before strdup */
+			engine_colon = strchr(engine_str, ':');
 
 			*engine_colon = '\0';		/* engine_str now has engine name */
 			engine_colon++;		/* engine_colon now has key name */