summaryrefslogtreecommitdiffstats
path: root/amqp-pr274.patch
blob: 48174706fc5724d3946b874611cc93e72cc3ff51 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
From f031d8492f0acfc0f38e8e5e4e01ed87a22234b2 Mon Sep 17 00:00:00 2001
From: Remi Collet <fedora@famillecollet.com>
Date: Mon, 13 Mar 2017 06:57:14 +0100
Subject: [PATCH] adapt buffer size and secure its usage, fix #273

---
 amqp_timestamp.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/amqp_timestamp.c b/amqp_timestamp.c
index 70013f1..79b0e52 100644
--- a/amqp_timestamp.c
+++ b/amqp_timestamp.c
@@ -119,7 +119,7 @@ zend_function_entry amqp_timestamp_class_functions[] = {
 PHP_MINIT_FUNCTION(amqp_timestamp)
 {
 	zend_class_entry ce;
-	char min[1], max[20];
+	char min[20], max[20];
 	int min_len, max_len;
 
 	INIT_CLASS_ENTRY(ce, "AMQPTimestamp", amqp_timestamp_class_functions);
@@ -128,10 +128,10 @@ PHP_MINIT_FUNCTION(amqp_timestamp)
 
 	zend_declare_property_null(this_ce, ZEND_STRL("timestamp"), ZEND_ACC_PRIVATE TSRMLS_CC);
 
-	max_len = sprintf(max, "%.0f", AMQP_TIMESTAMP_MAX);
+	max_len = snprintf(max, sizeof(max), "%.0f", AMQP_TIMESTAMP_MAX);
 	zend_declare_class_constant_stringl(this_ce, ZEND_STRL("MAX"), max, max_len TSRMLS_CC);
 
-	min_len = sprintf(min, "%.0f", AMQP_TIMESTAMP_MIN);
+	min_len = snprintf(min, sizeof(min), "%.0f", AMQP_TIMESTAMP_MIN);
 	zend_declare_class_constant_stringl(this_ce, ZEND_STRL("MIN"), min, min_len TSRMLS_CC);
 
 	return SUCCESS;