regtool support for remote registry
Hi! Attached patch allows regtool to access registry on remote hosts, e.g.: regtool get \\bumba\machine\software\microsoft\windows\currentVersion\programFilesDir egor.mailto:[EMAIL PROTECTED] icq 5165414 fidonet 2:5020/496.19 regtool-remote-registry-support.diff Description: Binary data regtool-remote-registry-support.ChangeLog Description: Binary data
Re: Name aliasing in security.cc
On Wed, Jun 05, 2002 at 08:23:59PM -0400, Pierre A. Humblet wrote: I saw the changes in grp.cc and passwd.cc where you make default entries from the token. That's a good idea, very close to what I had in mind for the except clause in suggestion c) above. At any rate this doesn't favor keeping lookup_name() and using it up only in alloc_sd(). So you could still apply my patches, even if you want to move from b) to the direction of c). Ok, applied. However, I think calling lookup_name in internal_getlogin() is somewhat useless. I agree. My patches remove it, but replace it with something similar. I will remove it later if you apply them. By the way, your ChangeLog entry is missing * passwd.cc :) :) :) Thanks, I've fixed that. Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Developermailto:[EMAIL PROTECTED] Red Hat, Inc.
Re: Patch for sub-second resolution in stat(2)
On Thu, Jun 06, 2002 at 01:26:57PM +0100, Conrad Scott wrote: Oh sh*t. It turns out that my changelog entry was more broken than just formatting. I've appended and attached a new one to this email (the attachment is because I'm unclear that Outlook Express keeps the formatting). Sorry for imposing my sordid learning experience on everyone. // Conrad 2002-06-05 Conrad Scott [EMAIL PROTECTED] * fhandler.cc (fhandler_base::fstat): Initialise tv_nsec member of st_atim, st_mtim, and st_ctim fields. * fhandler_disk_file.cc (fhandler_disk_file::fstat_helper): Ditto. * fhandler_process.cc (fhandler_process::fstat): Ditto. * glob.c (stat32_to_STAT): Copy across the whole st_atim, st_mtime, and st_ctim fields. * syscalls.cc (stat64_to_stat32): Ditto. * times.cc (to_timestruc_t): New function. (time_as_timestruc_t): New function. * winsup.h: Add to_timestruc_t and time_as_timestruc_t functions. * include/cygwin/stat.h: Replace time_t with timestruc_t throughout for all file times, removing the st_spare1, st_spare2, and st_spare3 fields in the process. Add macros to access tv_sec fields by old names. ^^ If you now substitute the leading space by a leading TAB, the ChangeLog entry would be correct. ;-) Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Developermailto:[EMAIL PROTECTED] Red Hat, Inc.
Re: Patch for sub-second resolution in stat(2)
The leading tab is in the attached file. His mail client converted it to a leading space. Earnie. Corinna Vinschen wrote: ^^ If you now substitute the leading space by a leading TAB, the ChangeLog entry would be correct. ;-) Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Developermailto:[EMAIL PROTECTED] Red Hat, Inc.
Re: regtool support for remote registry
On Thu, Jun 06, 2002 at 02:11:46PM +0400, Egor Duda wrote: Hi! Attached patch allows regtool to access registry on remote hosts, e.g.: regtool get \\bumba\machine\software\microsoft\windows\currentVersion\programFilesDir Just a question: Wouldn't it make sense to use a syntax as rcp/scp: regtool get bumba:\machine\software\microsoft\... This together with the capability to use forward slashes looks more unixy: regtool get bumba:/machine/software/microsoft/... In this context I see for the first time, that the usage doesn't say anything about using forward slashes. Shouldn't we add a word that this is possible, too? Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Developermailto:[EMAIL PROTECTED] Red Hat, Inc.
Re: Patch for sub-second resolution in stat(2)
On Thu, Jun 06, 2002 at 02:25:33PM +0100, Conrad Scott wrote: Corinna Vinschen [EMAIL PROTECTED] wrote: If you now substitute the leading space by a leading TAB, the ChangeLog entry would be correct. ;-) Corinna But I can't convince Outlook Express to insert tab characters into a message! (breaks down sobbing with the utter futility of using M$ products). Ouch. So, time to start using some other email agent. I hope everyone is waiting with baited breath to see what I get wrong then :-) What about mutt? Or at least Mozilla? Today I had a call from a friend to help with Outlook Express. Oh, my! I'm really not trying to pick onto MS but OE is not one of the good pitches... Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Developermailto:[EMAIL PROTECTED] Red Hat, Inc.
Re: Patch for sub-second resolution in stat(2)
Conrad Scott wrote: Corinna Vinschen [EMAIL PROTECTED] wrote: If you now substitute the leading space by a leading TAB, the ChangeLog entry would be correct. ;-) Corinna But I can't convince Outlook Express to insert tab characters into a message! (breaks down sobbing with the utter futility of using M$ products). So, time to start using some other email agent. I hope everyone is waiting with baited breath to see what I get wrong then :-) Attach the ChangeLog entry to the mail, make sure the mime for the attachment is text or make sure the file has a .txt suffix. Most mail clients will then display the attachment within the body of the mail. Earnie.
Re: Name aliasing in security.cc
Corinna Vinschen wrote: On Wed, Jun 05, 2002 at 08:23:59PM -0400, Pierre A. Humblet wrote: I saw the changes in grp.cc and passwd.cc where you make default entries from the token. That's a good idea, very close to what I had in mind for the except clause in suggestion c) above. Corinna, Something came to my mind last night: the sid that your new code in passwd.cc gets from the token should already be in user.sid (and user.orig_sid). Why not get it from there instead? I also have a question: why does internal Cygwin code use strcasematch() instead of !strcasecmp()? I (ignorantly) used strcasecmp() recently, it seems to work. Pierre
Re: Patch for sub-second resolution in stat(2)
You know, I had some trouble with this, too. Maybe it would be worth adding to contrib.html that attaching a ChangeLog.txt file works well to preserve formatting? --- Conrad Scott [EMAIL PROTECTED] wrote: Earnie Boyd [EMAIL PROTECTED] wrote: Attach the ChangeLog entry to the mail, make sure the mime for the attachment is text or make sure the file has a .txt suffix. Most mail clients will then display the attachment within the body of the mail. Thanks for the hint: I've just had a look and yes, OE was attaching the changelog file as 'application/octet-stream' (i.e. binary). I'll try saving the file as 'changelog.txt' and attaching that next time. Thanks. // Conrad __ Do You Yahoo!? Yahoo! - Official partner of 2002 FIFA World Cup http://fifaworldcup.yahoo.com
Re: Name aliasing in security.cc
Corinna Vinschen wrote: You're right but it doesn't matter since I have to open the token anyway since I need the primary group which isn't available at that point. Never mind. I now understand what you mean. Still the whole thing (uinfo, passwd, grp) can probably be optimized. Pierre
Add new -T|--toggle option to strace
As per a discussion yesterday with Chris, I've added a new option to strace, -T or --toggle, to toggle tracing in a process already being straced. One other change I made was to replace a use of ExitProcess() with exit() --- otherwise some error messages didn't appear since the stdio buffers weren't being flushed. I've also updated the section for strace in utils.sgml. I know that Joshua has been holding off on changing utils.sgml (see http://sources.redhat.com/ml/cygwin-patches/2002-q2/msg00276.html) so I don't know whether you want this update or not. It's not much more than a cut and paste from the usage message in strace.cc. ChangeLog appended here and attached (to keep its format out of Outlook's clutches). // Conrad 2002-06-07 Conrad Scott [EMAIL PROTECTED] * strace.cc (toggle): New global variable. (error): Use exit instead of ExitProcess so that stdio buffers get flushed. (create_child): Remove command line error checking. (dostrace): Ditto. (dotoggle): New function. (usage): Add entry for new option -T|--toggle. Alphabetize. (longopts): Add new option -T|--toggle. (opts): Ditto. (main): Handle new -T|--toggle option. Move all command line checking here from other functions. * utils.sgml: Update section for strace. strace.patch Description: Binary data utils.patch Description: Binary data 2002-06-07 Conrad Scott [EMAIL PROTECTED] * strace.cc (toggle): New global variable. (error): Use exit instead of ExitProcess so that stdio buffers get flushed. (create_child): Remove command line error checking. (dostrace): Ditto. (dotoggle): New function. (usage): Add entry for new option -T|--toggle. Alphabetize. (longopts): Add new option -T|--toggle. (opts): Ditto. (main): Handle new -T|--toggle option. Move all command line checking here from other functions. * utils.sgml: Update section for strace.
Re: Add new -T|--toggle option to strace
On Fri, Jun 07, 2002 at 02:09:17AM +0100, Conrad Scott wrote: 2002-06-07 Conrad Scott [EMAIL PROTECTED] * strace.cc (toggle): New global variable. (error): Use exit instead of ExitProcess so that stdio buffers get flushed. (create_child): Remove command line error checking. (dostrace): Ditto. (dotoggle): New function. (usage): Add entry for new option -T|--toggle. Alphabetize. (longopts): Add new option -T|--toggle. (opts): Ditto. (main): Handle new -T|--toggle option. Move all command line checking here from other functions. * utils.sgml: Update section for strace. Applied. Thanks, cgf