public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
* (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 12:21               ` Robert Mark
@ 2012-09-12 13:33                 ` Andrey Repin
  2012-09-12 21:33                   ` Andrey Repin
  0 siblings, 1 reply; 14+ messages in thread
From: Andrey Repin @ 2012-09-12 13:33 UTC (permalink / raw)
  To: Robert Mark, cygwin

Greetings, Robert Mark!

> 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!

Try adding \0 instead.


--
WBR,
Andrey Repin (anrdaemon@freemail.ru) 12.09.2012, <17:01>

Sorry for my terrible english...


--
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

* Re: (core dumped) mkshortcut
  2012-09-12 13:33                 ` Andrey Repin
@ 2012-09-12 21:33                   ` Andrey Repin
  0 siblings, 0 replies; 14+ messages in thread
From: Andrey Repin @ 2012-09-12 21:33 UTC (permalink / raw)
  To: Andrey Repin

Greetings, Andrey Repin!

>> 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!

> Try adding \0 instead.

Actually, disregard this suggestion... Sorry for noise.



--
WBR,
Andrey Repin (anrdaemon@freemail.ru) 12.09.2012, <22:51>

Sorry for my terrible english...


--
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).