Skip to content

Commit 4f5838a

Browse files
committed
replace sprintf with snprintf
1 parent 3a23648 commit 4f5838a

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

middle-pgsql.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -152,14 +152,14 @@ pg_result_t middle_query_pgsql_t::exec_prepared(char const *stmt,
152152
osmid_t osm_id) const
153153
{
154154
char buffer[64];
155-
sprintf(buffer, "%" PRIdOSMID, osm_id);
155+
snprintf(buffer, sizeof(buffer), "%" PRIdOSMID, osm_id);
156156
return exec_prepared(stmt, buffer);
157157
}
158158

159159
pg_result_t middle_pgsql_t::exec_prepared(char const *stmt, osmid_t osm_id) const
160160
{
161161
char buffer[64];
162-
sprintf(buffer, "%" PRIdOSMID, osm_id);
162+
snprintf(buffer, sizeof(buffer), "%" PRIdOSMID, osm_id);
163163
auto bptr = const_cast<char const *>(buffer);
164164
return pgsql_execPrepared(m_query_conn, stmt, 1, &bptr, PGRES_TUPLES_OK);
165165
}

output-gazetteer.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ void output_gazetteer_t::delete_unused_classes(char const *osm_type,
2323
}
2424

2525
char id[64];
26-
sprintf(id, "%" PRIdOSMID, osm_id);
26+
snprintf(id, sizeof(id), "%" PRIdOSMID, osm_id);
2727
char const *params[2] = {osm_type, id};
2828

2929
auto res = pgsql_execPrepared(m_conn, "get_classes", 2, params,
@@ -53,7 +53,7 @@ void output_gazetteer_t::delete_unused_classes(char const *osm_type,
5353
void output_gazetteer_t::delete_place(char const *osm_type, osmid_t osm_id)
5454
{
5555
char id[64];
56-
sprintf(id, "%" PRIdOSMID, osm_id);
56+
snprintf(id, sizeof(id), "%" PRIdOSMID, osm_id);
5757
char const *params[2] = {osm_type, id};
5858

5959
pgsql_execPrepared(m_conn, "delete_place", 2, params, PGRES_COMMAND_OK);

0 commit comments

Comments
 (0)