We malloc(len + 1), so strlcpy(..., len + 1), rather than ... len

diff --git a/src/modules/rlm_sql/drivers/rlm_sql_postgresql/sql_postgresql.c b/src/modules/rlm_sql/drivers/rlm_sql_postgresql/sql_postgresql.c
index ca0efb3..eb09538 100644 (file)
--- a/src/modules/rlm_sql/drivers/rlm_sql_postgresql/sql_postgresql.c
+++ b/src/modules/rlm_sql/drivers/rlm_sql_postgresql/sql_postgresql.c
@@ -377,7 +377,7 @@ static int sql_fetch_row(SQLSOCK * sqlsocket, UNUSED SQL_CONFIG *config) {
                        len = PQgetlength(pg_sock->result, pg_sock->cur_row, i);
                        pg_sock->row[i] = (char *)rad_malloc(len+1);
                        memset(pg_sock->row[i], '\0', len+1);
-                       strlcpy(pg_sock->row[i], PQgetvalue(pg_sock->result, pg_sock->cur_row,i),len);
+                       strlcpy(pg_sock->row[i], PQgetvalue(pg_sock->result, pg_sock->cur_row,i),len + 1);
                }
                pg_sock->cur_row++;
                sqlsocket->row = pg_sock->row;