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,

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

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

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

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

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 stdio.h #include

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 stdio.h #include gio/gio.h void gfile_add_watch (const char *file) { GFile *gfile =

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

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 stdio.h #include gio/gio.h void gfile_add_watch (const char

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 stdio.h #include gio/gio.h void gfile_add_watch (const char *file) { GFile *gfile = g_file_new_for_path (file); GFileMonitor *monitor;