public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
* crash on environment name beginning with non-ASCII char (64 bit)
@ 2017-05-21  2:04 Thomas Wolff
  2017-05-21  2:07 ` Steven Penny
  2017-06-06 19:15 ` Corinna Vinschen
  0 siblings, 2 replies; 7+ messages in thread
From: Thomas Wolff @ 2017-05-21  2:04 UTC (permalink / raw)
  To: cygwin

If an environment variable is set in the Windows environment that has a 
name beginning with a non-ASCII character (e.g. "äx", value "test"),
initialising the 64-bit cygwin dll fails:
*** fatal error - internal error reading the windows environment - too 
many environment variables?
No problem with the 32-bit dll. Also 64-bit works e.g. with a variable 
name of "aäx".
Thomas

--
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] 7+ messages in thread

* Re: crash on environment name beginning with non-ASCII char (64 bit)
  2017-05-21  2:04 crash on environment name beginning with non-ASCII char (64 bit) Thomas Wolff
@ 2017-05-21  2:07 ` Steven Penny
  2017-05-21  2:12   ` Steven Penny
  2017-06-06 19:15 ` Corinna Vinschen
  1 sibling, 1 reply; 7+ messages in thread
From: Steven Penny @ 2017-05-21  2:07 UTC (permalink / raw)
  To: cygwin

On Sat, 20 May 2017 22:04:21, Thomas Wolff wrote:
> If an environment variable is set in the Windows environment that has a 
> name beginning with a non-ASCII character (e.g. "äx", value "test"),
> initialising the 64-bit cygwin dll fails:
> *** fatal error - internal error reading the windows environment - too 
> many environment variables?
> No problem with the 32-bit dll. Also 64-bit works e.g. with a variable 
> name of "aäx".

At first I assumed this was the ongoing ReadLine problem:

http://cygwin.com/ml/cygwin/2017-04/msg00176.html

in which case I would ask you to see if the problem persists with Dash, or if it
is exclusive to Bash. However I am not able to repro your issue. I tried it in
pristine Windows 7 virtual machine with:

- Cygwin desktop shortcut
- Cygwin.bat
- Directory\Background\shell reg key

However, even though I cannot repro this, it still seems like a ReadLine issue,
as it seems too similar to a similar problem I am having to be a coincidence:

http://cygwin.com/ml/cygwin/2017-02/msg00031.html


--
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] 7+ messages in thread

* Re: crash on environment name beginning with non-ASCII char (64 bit)
  2017-05-21  2:07 ` Steven Penny
@ 2017-05-21  2:12   ` Steven Penny
  2017-05-21  4:25     ` Steven Penny
  0 siblings, 1 reply; 7+ messages in thread
From: Steven Penny @ 2017-05-21  2:12 UTC (permalink / raw)
  To: cygwin

On Sat, 20 May 2017 19:04:31, Steven Penny wrote:
> in which case I would ask you to see if the problem persists with Dash, or if it
> is exclusive to Bash. However I am not able to repro your issue. I tried it in
> pristine Windows 7 virtual machine with:

Crap, I was confused by your wording. *I can* repro this if the variable *name*
begins with non-ASCII.


--
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] 7+ messages in thread

* Re: crash on environment name beginning with non-ASCII char (64 bit)
  2017-05-21  2:12   ` Steven Penny
@ 2017-05-21  4:25     ` Steven Penny
  0 siblings, 0 replies; 7+ messages in thread
From: Steven Penny @ 2017-05-21  4:25 UTC (permalink / raw)
  To: cygwin

On Sat, 20 May 2017 19:07:41, Steven Penny wrote:
> Crap, I was confused by your wording. *I can* repro this if the variable
> *name* begins with non-ASCII.

Sadly, this appears to affect Dash as well, so it goes all the way down to the
Cygwin DLL, as you said.


--
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] 7+ messages in thread

* Re: crash on environment name beginning with non-ASCII char (64 bit)
  2017-05-21  2:04 crash on environment name beginning with non-ASCII char (64 bit) Thomas Wolff
  2017-05-21  2:07 ` Steven Penny
@ 2017-06-06 19:15 ` Corinna Vinschen
  2017-06-06 21:11   ` Thomas Wolff
  2017-06-18 15:27   ` Steven Penny
  1 sibling, 2 replies; 7+ messages in thread
From: Corinna Vinschen @ 2017-06-06 19:15 UTC (permalink / raw)
  To: cygwin

[-- Attachment #1: Type: text/plain, Size: 639 bytes --]

On May 20 22:04, Thomas Wolff wrote:
> If an environment variable is set in the Windows environment that has a name
> beginning with a non-ASCII character (e.g. "äx", value "test"),
> initialising the 64-bit cygwin dll fails:
> *** fatal error - internal error reading the windows environment - too many
> environment variables?

Thanks for th report.  I applied a patch and uploaded a new developer
snapshot to https://cygwin.com/snapshots/.  Please test.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: crash on environment name beginning with non-ASCII char (64 bit)
  2017-06-06 19:15 ` Corinna Vinschen
@ 2017-06-06 21:11   ` Thomas Wolff
  2017-06-18 15:27   ` Steven Penny
  1 sibling, 0 replies; 7+ messages in thread
From: Thomas Wolff @ 2017-06-06 21:11 UTC (permalink / raw)
  To: cygwin

Am 06.06.2017 um 21:15 schrieb Corinna Vinschen:
> On May 20 22:04, Thomas Wolff wrote:
>> If an environment variable is set in the Windows environment that has a name
>> beginning with a non-ASCII character (e.g. "äx", value "test"),
>> initialising the 64-bit cygwin dll fails:
>> *** fatal error - internal error reading the windows environment - too many
>> environment variables?
> Thanks for th report.  I applied a patch and uploaded a new developer
> snapshot to https://cygwin.com/snapshots/.  Please test.
Great, thanks!
Thomas

--
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] 7+ messages in thread

* Re: crash on environment name beginning with non-ASCII char (64 bit)
  2017-06-06 19:15 ` Corinna Vinschen
  2017-06-06 21:11   ` Thomas Wolff
@ 2017-06-18 15:27   ` Steven Penny
  1 sibling, 0 replies; 7+ messages in thread
From: Steven Penny @ 2017-06-18 15:27 UTC (permalink / raw)
  To: cygwin

On Tue, 6 Jun 2017 21:15:30, Corinna Vinschen wrote:
> Thanks for th report.  I applied a patch and uploaded a new developer
> snapshot to https://cygwin.com/snapshots/.  Please test.

Confirmed fixed


--
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] 7+ messages in thread

end of thread, other threads:[~2017-06-18 15:27 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-21  2:04 crash on environment name beginning with non-ASCII char (64 bit) Thomas Wolff
2017-05-21  2:07 ` Steven Penny
2017-05-21  2:12   ` Steven Penny
2017-05-21  4:25     ` Steven Penny
2017-06-06 19:15 ` Corinna Vinschen
2017-06-06 21:11   ` Thomas Wolff
2017-06-18 15:27   ` Steven Penny

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