> On 29 Dec 2017, at 16:56, Torsten Bögershausen <tboegi@xxxxxx> wrote:
>
> On Fri, Dec 29, 2017 at 04:22:18PM +0100, lars.schneider@xxxxxxxxxxxx wrote:
>> From: Lars Schneider <larsxschneider@xxxxxxxxx>
>>
>> Create a copy of an existing string and make all characters upper case.
>> Similar xstrdup_tolower().
>>
>> This function is used in a subsequent commit.
>>
>> Signed-off-by: Lars Schneider <larsxschneider@xxxxxxxxx>
>> ---
>> strbuf.c | 13 +++++++++++++
>> strbuf.h | 1 +
>> 2 files changed, 14 insertions(+)
>>
>> diff --git a/strbuf.c b/strbuf.c
>> index 323c49ceb3..54276e96e7 100644
>> --- a/strbuf.c
>> +++ b/strbuf.c
>> @@ -760,6 +760,19 @@ char *xstrdup_tolower(const char *string)
>> return result;
>> }
>>
>> +char *xstrdup_toupper(const char *string)
>> +{
>> + char *result;
>> + size_t len, i;
>> +
>> + len = strlen(string);
>> + result = xmallocz(len);
>> + for (i = 0; i < len; i++)
>> + result[i] = toupper(string[i]);
>> + result[i] = '\0';
> ^^^^^^^^^^^^^^^^
> Isn't this already done by xmallocz()
I copied that code from xstrdup_tolower().
The original implementation [1] and its refactored version [2]
used xmalloc(). Later on xmallocz [3] was introduced.
[3] states "we can stop manually placing NUL at the end of the
allocated buffer. But that's only safe if it's clear that
the contents will always fill the buffer."
As far as I understand it, the content should always fill the
buffer in the upper/lower case conversion. Therefore, I agree
with you that the assignment is not necessary.
- Lars
[1] d4770964d5 (config: "git config --get-urlmatch" parses section.<url>.key, 2013-07-31)
[2] 88d5a6f6cd (daemon/config: factor out duplicate xstrdup_tolower, 2014-05-22)
[3] 3733e69464 (use xmallocz to avoid size arithmetic, 2016-02-22)
