summaryrefslogtreecommitdiffstats
path: root/libc/stdio
diff options
context:
space:
mode:
authorMukesh Ojha <mukesh02@linux.vnet.ibm.com>2016-09-02 10:26:39 +0530
committerStewart Smith <stewart@linux.vnet.ibm.com>2016-10-11 15:40:30 +1100
commit32dc2a19bb6a91862b414fdd6eec07154e5e4308 (patch)
tree4de14dfaf6180b4a7c494c9e5a9dee230d7c11ed /libc/stdio
parentedf57684bd6821d4d01515b6e599f477e04ca01d (diff)
downloadblackbird-skiboot-32dc2a19bb6a91862b414fdd6eec07154e5e4308.tar.gz
blackbird-skiboot-32dc2a19bb6a91862b414fdd6eec07154e5e4308.zip
libc : Changes variable data type where return value of strlen() stored from int to size_t
Reason of the change as integer value may overflow, and it can give negative value for the length. This patch also changes the data type of variable which is compared with strlen() as the comparison also has to be done on the same level. Signed-off-by: Mukesh Ojha <mukesh02@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'libc/stdio')
-rw-r--r--libc/stdio/vsnprintf.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/libc/stdio/vsnprintf.c b/libc/stdio/vsnprintf.c
index fbb84a0b..38d52818 100644
--- a/libc/stdio/vsnprintf.c
+++ b/libc/stdio/vsnprintf.c
@@ -26,7 +26,7 @@ static int
print_str_fill(char **buffer, size_t bufsize, char *sizec,
const char *str, char c)
{
- int i, sizei, len;
+ size_t i, sizei, len;
char *bstart = *buffer;
sizei = strtoul(sizec, NULL, 10);
@@ -46,7 +46,7 @@ static int
print_str(char **buffer, size_t bufsize, const char *str)
{
char *bstart = *buffer;
- int i;
+ size_t i;
for (i = 0; (i < strlen(str)) && ((*buffer - bstart) < bufsize); i++) {
**buffer = str[i];
OpenPOWER on IntegriCloud