(core dumped) mkshortcut

[Peter Rosin]

On 2012-09-11 10:24, marco atzeri wrote:
> On 9/11/2012 9:59 AM, Pawel Jasinski wrote:
>> hi,
>>
>>>>>>>> mkshortcut -n "I for cygwin bin" /D/cyghome/bin
>>>>>>>> /home/RobertMarkBram/bin/createWindowsShortcut.sh: line 160:  8128
>>>>>>>> Aborted                 (core dumped) mkshortcut -n "$shortcutName"
>>>>>>>> "$target"
>>>>>>>
>>
>>
>> I confirm that, there is something fishy here.
>>
>> this one is from XP
>>
>> REJAP at rzuem5008 ~
>> $ mkshortcut -n "Ifor cygwin bin" /c/cygwin/bin
>>
>> REJAP at rzuem5008 ~
>> $ mkshortcut -n "I for cygwin bin" /c/cygwin/bin
>> Aborted (core dumped)
>>
>>
>> and this one out of Win7 32
>>
>> rejap at win7dev ~
>> $ mkshortcut -n "I for cygwin bin" /c/cygwin/bin
>> Aborted (core dumped)
>>
>> rejap at win7dev ~
>> $ mkshortcut -n "Ifor cygwin bin" /c/cygwin/bin
>>
>>
>> What is interesting, both links lnk file exist and are identical.
>>
>>
>> Cheers,
>> Pawel
>>
> 
> Pawel,
> your cases are the same. I had same result on W7 64
> It seems depending on shortcut length, when multiple of 16:
> 
> marco@MARCOATZERI ~
> $ mkshortcut -n '1234567890123456' '/e/cygwin/e'
> Aborted (core dumped)
> 
> marco@MARCOATZERI ~
> $ mkshortcut -n '12345678901234567890123456789012' '/e/cygwin/e'
> Aborted (core dumped)
> 
> a longer or shorter name works.
> 
> marco@MARCOATZERI ~
> $ mkshortcut -n '123456789012345678901234567890123' '/e/cygwin/e'


Here's a patch that fixes the bug.

Cheers,
Peter

--- src/mkshortcut/mkshortcut.c.orig	2012-04-13 05:12:18.000000000 +0200
+++ src/mkshortcut/mkshortcut.c	2012-09-12 14:46:02.962381900 +0200
@@ -395,7 +395,7 @@
 xstrncat (char **dest, const char *add, size_t n)
 {
   size_t len = strlen (*dest) + n;
-  char *s = (char *) realloc (*dest, len * sizeof (char));
+  char *s = (char *) realloc (*dest, len * sizeof (char) + 1);
   if (!s)
   {
     fprintf (stderr, "%s: out of memory\n", program_name);


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple