* (core dumped) mkshortcut
@ 2012-09-09 4:43 Robert Mark
2012-09-09 17:34 ` Earnie Boyd
0 siblings, 1 reply; 14+ messages in thread
From: Robert Mark @ 2012-09-09 4:43 UTC (permalink / raw)
To: cygwin
Hi All,
As per a previous message
(http://cygwin.com/ml/cygwin/2012-05/msg00215.html) I am still finding
that mkshortcut causes core dumps (Windows 7, 64 bit). I have tried a
few different options: -P or -D still cause dumps. I have checked
permissions and they are all ok. Some folders do cause the error, some
don't.
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 am using an alternative as suggested:
http://cygwin.com/ml/cygwin/2012-05/msg00232.html
It works OK, but isn't nearly as useful as mkshortcut: lnk files
handle rather differently than url files.
Kind Regards,
Rob
:)
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (core dumped) mkshortcut
2012-09-09 4:43 (core dumped) mkshortcut Robert Mark
@ 2012-09-09 17:34 ` Earnie Boyd
2012-09-10 11:59 ` Robert Mark
0 siblings, 1 reply; 14+ messages in thread
From: Earnie Boyd @ 2012-09-09 17:34 UTC (permalink / raw)
To: cygwin
On Sun, Sep 9, 2012 at 12:07 AM, Robert Mark wrote:
> Hi All,
>
> As per a previous message
> (http://cygwin.com/ml/cygwin/2012-05/msg00215.html) I am still finding
> that mkshortcut causes core dumps (Windows 7, 64 bit). I have tried a
> few different options: -P or -D still cause dumps. I have checked
> permissions and they are all ok. Some folders do cause the error, some
> don't.
>
> 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'm going to guess but have no fact to back it that the reason is due
to the fact you're executing in a 32bit environment and the shortcut
the tool makes needs a 64bit enviroment. This may only be resolved by
a 64bit version of Cygwin.
>
> I am using an alternative as suggested:
> http://cygwin.com/ml/cygwin/2012-05/msg00232.html
>
> It works OK, but isn't nearly as useful as mkshortcut: lnk files
> handle rather differently than url files.
You can still automate it. You just have to create a text file
instead of a binary one.
--
Earnie
-- https://sites.google.com/site/earnieboyd
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (core dumped) mkshortcut
2012-09-09 17:34 ` Earnie Boyd
@ 2012-09-10 11:59 ` Robert Mark
2012-09-10 13:00 ` Earnie Boyd
0 siblings, 1 reply; 14+ messages in thread
From: Robert Mark @ 2012-09-10 11:59 UTC (permalink / raw)
To: cygwin
Hi Earnie,
>> As per a previous message
>> (http://cygwin.com/ml/cygwin/2012-05/msg00215.html) I am still finding
>> that mkshortcut causes core dumps (Windows 7, 64 bit). I have tried a
>> few different options: -P or -D still cause dumps. I have checked
>> permissions and they are all ok. Some folders do cause the error, some
>> don't.
>>
>> 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'm going to guess but have no fact to back it that the reason is due
> to the fact you're executing in a 32bit environment and the shortcut
> the tool makes needs a 64bit enviroment. This may only be resolved by
> a 64bit version of Cygwin.
I did say I was running 64bit Win 7, so what exactly do you mean?
Cygwin.com says "The Cygwin DLL currently works with all recent,
commercially released x86 32 bit and 64 bit versions of Windows".
>> I am using an alternative as suggested:
>> http://cygwin.com/ml/cygwin/2012-05/msg00232.html
>>
>> It works OK, but isn't nearly as useful as mkshortcut: lnk files
>> handle rather differently than url files.
>
> You can still automate it. You just have to create a text file
> instead of a binary one.
I will try that - thanks.
Rob
:)
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (core dumped) mkshortcut
2012-09-10 11:59 ` Robert Mark
@ 2012-09-10 13:00 ` Earnie Boyd
2012-09-10 19:20 ` Denis Excoffier
0 siblings, 1 reply; 14+ messages in thread
From: Earnie Boyd @ 2012-09-10 13:00 UTC (permalink / raw)
To: cygwin
On Mon, Sep 10, 2012 at 7:02 AM, Robert Mark wrote:
> Hi Earnie,
>
>>> As per a previous message
>>> (http://cygwin.com/ml/cygwin/2012-05/msg00215.html) I am still finding
>>> that mkshortcut causes core dumps (Windows 7, 64 bit). I have tried a
>>> few different options: -P or -D still cause dumps. I have checked
>>> permissions and they are all ok. Some folders do cause the error, some
>>> don't.
>>>
>>> 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'm going to guess but have no fact to back it that the reason is due
>> to the fact you're executing in a 32bit environment and the shortcut
>> the tool makes needs a 64bit enviroment. This may only be resolved by
>> a 64bit version of Cygwin.
>
> I did say I was running 64bit Win 7, so what exactly do you mean?
>
I mean that Cygwin at the moment is a 32 bit application. So you are
executing in an emulation mode and some things might just not work as
expected.
> Cygwin.com says "The Cygwin DLL currently works with all recent,
> commercially released x86 32 bit and 64 bit versions of Windows".
>
It works but might not be able to do everything that is required due
to the limitations of the emulated environment.
--
Earnie
-- https://sites.google.com/site/earnieboyd
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (core dumped) mkshortcut
2012-09-10 13:00 ` Earnie Boyd
@ 2012-09-10 19:20 ` Denis Excoffier
2012-09-11 5:58 ` Earnie Boyd
0 siblings, 1 reply; 14+ messages in thread
From: Denis Excoffier @ 2012-09-10 19:20 UTC (permalink / raw)
To: Earnie Boyd; +Cc: cygwin
On 2012-09-10 at 14:54, Earnie Boyd wrote:
> On Mon, Sep 10, 2012 at 7:02 AM, Robert Mark wrote:
>> Hi Earnie,
>>
>>>> As per a previous message
>>>> (http://cygwin.com/ml/cygwin/2012-05/msg00215.html) I am still finding
>>>> that mkshortcut causes core dumps (Windows 7, 64 bit). I have tried a
>>>> few different options: -P or -D still cause dumps. I have checked
>>>> permissions and they are all ok. Some folders do cause the error, some
>>>> don't.
>>>>
>>>> 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'm going to guess but have no fact to back it that the reason is due
>>> to the fact you're executing in a 32bit environment and the shortcut
>>> the tool makes needs a 64bit enviroment. This may only be resolved by
>>> a 64bit version of Cygwin.
>>
>> I did say I was running 64bit Win 7, so what exactly do you mean?
>>
>
> I mean that Cygwin at the moment is a 32 bit application. So you are
> executing in an emulation mode and some things might just not work as
> expected.
I must say that i run on 32 bits natively (XP) and have what i think is
the very same problem. See http://cygwin.com/ml/cygwin/2012-08/msg00542.html
I add here that my XP is "french only" (nationalization not possible).
Regards,
Denis Excoffier.
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (core dumped) mkshortcut
2012-09-10 19:20 ` Denis Excoffier
@ 2012-09-11 5:58 ` Earnie Boyd
2012-09-11 8:00 ` Pawel Jasinski
2012-09-11 8:25 ` Pawel Jasinski
0 siblings, 2 replies; 14+ messages in thread
From: Earnie Boyd @ 2012-09-11 5:58 UTC (permalink / raw)
To: cygwin
On Mon, Sep 10, 2012 at 3:16 PM, Denis Excoffier wrote:
>
> On 2012-09-10 at 14:54, Earnie Boyd wrote:
>> On Mon, Sep 10, 2012 at 7:02 AM, Robert Mark wrote:
>>> Hi Earnie,
>>>
>>>>> As per a previous message
>>>>> (http://cygwin.com/ml/cygwin/2012-05/msg00215.html) I am still finding
>>>>> that mkshortcut causes core dumps (Windows 7, 64 bit). I have tried a
>>>>> few different options: -P or -D still cause dumps. I have checked
>>>>> permissions and they are all ok. Some folders do cause the error, some
>>>>> don't.
>>>>>
>>>>> 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'm going to guess but have no fact to back it that the reason is due
>>>> to the fact you're executing in a 32bit environment and the shortcut
>>>> the tool makes needs a 64bit enviroment. This may only be resolved by
>>>> a 64bit version of Cygwin.
>>>
>>> I did say I was running 64bit Win 7, so what exactly do you mean?
>>>
>>
>> I mean that Cygwin at the moment is a 32 bit application. So you are
>> executing in an emulation mode and some things might just not work as
>> expected.
>
> I must say that i run on 32 bits natively (XP) and have what i think is
> the very same problem. See http://cygwin.com/ml/cygwin/2012-08/msg00542.html
> I add here that my XP is "french only" (nationalization not possible).
That is a horse of a different color; in your case I'm guessing that
mkshortcut is not supporting UNICODE file names but I don't know that.
--
Earnie
-- https://sites.google.com/site/earnieboyd
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (core dumped) mkshortcut
2012-09-11 5:58 ` Earnie Boyd
@ 2012-09-11 8:00 ` Pawel Jasinski
2012-09-11 8:25 ` Pawel Jasinski
1 sibling, 0 replies; 14+ messages in thread
From: Pawel Jasinski @ 2012-09-11 8:00 UTC (permalink / raw)
To: cygwin
hi,
I confirm that, there is something fishy here.
this one is from XP
REJAP@rzuem5008 ~
$ mkshortcut -n "Ifor cygwin bin" /c/cygwin/bin
REJAP@rzuem5008 ~
$ mkshortcut -n "I for cygwin bin" /c/cygwin/bin
Aborted (core dumped)
and this one out of Win7 32
rejap@win7dev ~
$ mkshortcut -n "I for cygwin bin" /c/cygwin/bin
Aborted (core dumped)
rejap@win7dev ~
$ mkshortcut -n "Ifor cygwin bin" /c/cygwin/bin
What is interesting, both links lnk file exist and are identical.
Cheers,
Pawel
On Mon, Sep 10, 2012 at 10:40 PM, Earnie Boyd
<earnie@users.sourceforge.net> wrote:
> On Mon, Sep 10, 2012 at 3:16 PM, Denis Excoffier wrote:
>>
>> On 2012-09-10 at 14:54, Earnie Boyd wrote:
>>> On Mon, Sep 10, 2012 at 7:02 AM, Robert Mark wrote:
>>>> Hi Earnie,
>>>>
>>>>>> As per a previous message
>>>>>> (http://cygwin.com/ml/cygwin/2012-05/msg00215.html) I am still finding
>>>>>> that mkshortcut causes core dumps (Windows 7, 64 bit). I have tried a
>>>>>> few different options: -P or -D still cause dumps. I have checked
>>>>>> permissions and they are all ok. Some folders do cause the error, some
>>>>>> don't.
>>>>>>
>>>>>> 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'm going to guess but have no fact to back it that the reason is due
>>>>> to the fact you're executing in a 32bit environment and the shortcut
>>>>> the tool makes needs a 64bit enviroment. This may only be resolved by
>>>>> a 64bit version of Cygwin.
>>>>
>>>> I did say I was running 64bit Win 7, so what exactly do you mean?
>>>>
>>>
>>> I mean that Cygwin at the moment is a 32 bit application. So you are
>>> executing in an emulation mode and some things might just not work as
>>> expected.
>>
>> I must say that i run on 32 bits natively (XP) and have what i think is
>> the very same problem. See http://cygwin.com/ml/cygwin/2012-08/msg00542.html
>> I add here that my XP is "french only" (nationalization not possible).
>
> That is a horse of a different color; in your case I'm guessing that
> mkshortcut is not supporting UNICODE file names but I don't know that.
>
> --
> Earnie
> -- https://sites.google.com/site/earnieboyd
>
> --
> 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
>
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (core dumped) mkshortcut
2012-09-11 5:58 ` Earnie Boyd
2012-09-11 8:00 ` Pawel Jasinski
@ 2012-09-11 8:25 ` Pawel Jasinski
2012-09-11 12:18 ` marco atzeri
1 sibling, 1 reply; 14+ messages in thread
From: Pawel Jasinski @ 2012-09-11 8:25 UTC (permalink / raw)
To: cygwin
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
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (core dumped) mkshortcut
2012-09-11 8:25 ` Pawel Jasinski
@ 2012-09-11 12:18 ` marco atzeri
2012-09-12 12:21 ` Robert Mark
2012-09-12 13:02 ` Peter Rosin
0 siblings, 2 replies; 14+ messages in thread
From: marco atzeri @ 2012-09-11 12:18 UTC (permalink / raw)
To: cygwin
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'
Regards
Marco
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (core dumped) mkshortcut
2012-09-11 12:18 ` marco atzeri
@ 2012-09-12 12:21 ` Robert Mark
2012-09-12 13:33 ` Andrey Repin
2012-09-12 13:02 ` Peter Rosin
1 sibling, 1 reply; 14+ messages in thread
From: Robert Mark @ 2012-09-12 12:21 UTC (permalink / raw)
To: cygwin
>> REJAP at rzuem5008 ~
>> $ mkshortcut -n "I for cygwin bin" /c/cygwin/bin
>> Aborted (core dumped)
>
> 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'
>
> Regards
> Marco
Brilliant!! Multiples of 8 is it - all my 8 and 16 character shortcut
names fail. Your powers of deduction are unparalleled Marco - thank
you! I now have a bunch of underscores in my shortcuts but at least my
script works!
Rob
:)
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (core dumped) mkshortcut
2012-09-11 12:18 ` marco atzeri
2012-09-12 12:21 ` Robert Mark
@ 2012-09-12 13:02 ` Peter Rosin
2012-09-12 18:58 ` Charles Wilson
1 sibling, 1 reply; 14+ messages in thread
From: Peter Rosin @ 2012-09-12 13:02 UTC (permalink / raw)
To: cygwin
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
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: (core dumped) mkshortcut
2012-09-12 13:02 ` Peter Rosin
@ 2012-09-12 18:58 ` Charles Wilson
0 siblings, 0 replies; 14+ messages in thread
From: Charles Wilson @ 2012-09-12 18:58 UTC (permalink / raw)
To: cygwin
On 9/12/2012 8:49 AM, Peter Rosin wrote:
> --- 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);
>
Thanks, Peter. I'll roll a new release within the next few days.
--
Chuck
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2012-09-12 19:05 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-09-09 4:43 (core dumped) mkshortcut Robert Mark
2012-09-09 17:34 ` Earnie Boyd
2012-09-10 11:59 ` Robert Mark
2012-09-10 13:00 ` Earnie Boyd
2012-09-10 19:20 ` Denis Excoffier
2012-09-11 5:58 ` Earnie Boyd
2012-09-11 8:00 ` Pawel Jasinski
2012-09-11 8:25 ` Pawel Jasinski
2012-09-11 12:18 ` marco atzeri
2012-09-12 12:21 ` Robert Mark
2012-09-12 13:33 ` Andrey Repin
2012-09-12 21:33 ` Andrey Repin
2012-09-12 13:02 ` Peter Rosin
2012-09-12 18:58 ` Charles Wilson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).