Re: Crash in g_file_monitor on 32-bit Cygwin

2014-10-15 Thread Ken Brown
On 10/15/2014 12:46 PM, Yaakov Selkowitz wrote: On 2014-10-14 13:30, Ken Brown wrote: I stepped into gam_check_not_fat (which I should have done to begin with) and narrowed this down further. The stack location in question gets clobbered by the call to GetVolumeInformation: [snip] Here's the

Re: Crash in g_file_monitor on 32-bit Cygwin

2014-10-15 Thread Yaakov Selkowitz
On 2014-10-14 13:30, Ken Brown wrote: I stepped into gam_check_not_fat (which I should have done to begin with) and narrowed this down further. The stack location in question gets clobbered by the call to GetVolumeInformation: [snip] Here's the code near the call to GetVolumeInformation, follo

Re: Crash in g_file_monitor on 32-bit Cygwin

2014-10-14 Thread Yaakov Selkowitz
On 2014-10-14 14:28, Corinna Vinschen wrote: I know the code is not yours, but I have to vent while I see this code :) Actually, this isn't the first time you're seeing this code, it's just been a while. :-) There's no reason to load GetVolumePathName from kernel32 since all supported platf

Re: Crash in g_file_monitor on 32-bit Cygwin

2014-10-14 Thread Corinna Vinschen
Hi Ken, I know the code is not yours, but I have to vent while I see this code :) On Oct 14 14:30, Ken Brown wrote: > I stepped into gam_check_not_fat (which I should have done to begin with) > and narrowed this down further. The stack location in question gets > clobbered by the call to GetVolu

Re: Crash in g_file_monitor on 32-bit Cygwin

2014-10-14 Thread Ken Brown
On 10/14/2014 12:26 PM, Ken Brown wrote: On 6/28/2014 7:08 AM, Ken Brown wrote: On 6/27/2014 1:52 PM, Yaakov Selkowitz wrote: On 2014-06-27 12:11, Ken Brown wrote: On 6/25/2014 10:17 PM, Ken Brown wrote: This is a followup to https://cygwin.com/ml/cygwin/2014-06/msg00324.html, from which I ex

Re: Crash in g_file_monitor on 32-bit Cygwin

2014-10-14 Thread Ken Brown
On 6/28/2014 7:08 AM, Ken Brown wrote: On 6/27/2014 1:52 PM, Yaakov Selkowitz wrote: On 2014-06-27 12:11, Ken Brown wrote: On 6/25/2014 10:17 PM, Ken Brown wrote: This is a followup to https://cygwin.com/ml/cygwin/2014-06/msg00324.html, from which I extracted the following test case: $ cat gf

Re: Crash in g_file_monitor on 32-bit Cygwin

2014-06-28 Thread Ken Brown
On 6/27/2014 1:52 PM, Yaakov Selkowitz wrote: On 2014-06-27 12:11, Ken Brown wrote: On 6/25/2014 10:17 PM, Ken Brown wrote: This is a followup to https://cygwin.com/ml/cygwin/2014-06/msg00324.html, from which I extracted the following test case: $ cat gfile-test.c #include #include void gfi

Re: Crash in g_file_monitor on 32-bit Cygwin

2014-06-27 Thread Yaakov Selkowitz
On 2014-06-27 12:11, Ken Brown wrote: On 6/25/2014 10:17 PM, Ken Brown wrote: This is a followup to https://cygwin.com/ml/cygwin/2014-06/msg00324.html, from which I extracted the following test case: $ cat gfile-test.c #include #include void gfile_add_watch (const char *file) { GFile *gfi

Re: Crash in g_file_monitor on 32-bit Cygwin

2014-06-27 Thread Ken Brown
On 6/27/2014 1:11 PM, Ken Brown wrote: My builds can be obtained from http://sanibeltranquility.com/cygwin/ if anyone else wants to try to reproduce this without rebuilding the packages themselves. I forgot to say that you'll probably have to run rebaseall after the install. Ken -- Pr

Re: Crash in g_file_monitor on 32-bit Cygwin

2014-06-27 Thread Ken Brown
On 6/25/2014 10:17 PM, Ken Brown wrote: This is a followup to https://cygwin.com/ml/cygwin/2014-06/msg00324.html, from which I extracted the following test case: $ cat gfile-test.c #include #include void gfile_add_watch (const char *file) { GFile *gfile = g_file_new_for_path (file); GF

Crash in g_file_monitor on 32-bit Cygwin

2014-06-25 Thread Ken Brown
This is a followup to https://cygwin.com/ml/cygwin/2014-06/msg00324.html, from which I extracted the following test case: $ cat gfile-test.c #include #include void gfile_add_watch (const char *file) { GFile *gfile = g_file_new_for_path (file); GFileMonitor *monitor; GFileMonitorFlags gfl