[SR-Dev] git:janakj/postgres: - another error condition fix for a problem that gets introduced by a

[Jan Janak]

Module: sip-router
Branch: janakj/postgres
Commit: 70764fb02a5be0f7ccca4f6e66adbdf083fdaea8
URL:    http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=70764fb02a5be0f7ccca4f6e66adbdf083fdaea8

Author: Henning Westerholt <henning.westerholt at 1und1.de>
Committer: Henning Westerholt <henning.westerholt at 1und1.de>
Date:   Fri Jan  9 13:23:11 2009 +0000

- another error condition fix for a problem that gets introduced by a
  recent change: don't try to free memory in the row_buf that don't
  belong to us anymore
- reported by Bayan Towfiq, bayan at flowroute dot com
- remove row_buf[col] NULL assignment that was not executed anyway


git-svn-id: https://openser.svn.sourceforge.net/svnroot/openser/trunk@5438 689a6050-402a-0410-94f2-e92a70836424

---

 modules/db_postgres/km_res.c |    9 ---------
 1 files changed, 0 insertions(+), 9 deletions(-)

diff --git a/modules/db_postgres/km_res.c b/modules/db_postgres/km_res.c
index 548e535..0cd80c5 100644
--- a/modules/db_postgres/km_res.c
+++ b/modules/db_postgres/km_res.c
@@ -256,19 +256,10 @@ int db_postgres_convert_rows(const db_con_t* _h, db_res_t* _r)
 		if(db_postgres_convert_row(_h, _r, &(RES_ROWS(_r)[row - RES_LAST_ROW(_r)]), row_buf)<0){
 			LM_ERR("failed to convert row #%d\n",  row);
 			RES_ROW_N(_r) = row - RES_LAST_ROW(_r);
-			for (col = 0; col < RES_COL_N(_r); col++) {
-				LM_DBG("freeing row_buf[%d] at %p\n", col, row_buf[col]);
-				pkg_free(row_buf[col]);
-			}
 			LM_DBG("freeing row buffer at %p\n", row_buf);
 			pkg_free(row_buf);
 			db_free_rows(_r);
 			return -4;
-			/*
-			 * The following housekeeping may not be technically required, but it
-			 * is a good practice to NULL pointer fields that are no longer valid.
-			 */
-			row_buf[col] = (char *)NULL;
 		}
 	}