Luc, I have to post an updated patch because I overlooked one case in my original test - the output redirection was left enabled after the sial command completed. It caused subsequent sial commands to be redirected (undesirable). This update saves and restores the original fp at the end of a redirection. Sorry... - Robert - Generated By: Teradata Corporation (Mon Nov 12 21:19:08 PST 2007) extensions.c | 12 +++++++++++++++++++++++++++++++++++++++++++++++++ extensions/sial.c | 13 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 25 insertions(+), 0 deletions(-) Index: crash-4.0-4.8/extensions/sial.c =================================================================== --- crash-4.0-4.8-orig/extensions/sial.c 2007-11-12 21:15:36.000000000 -0800 +++ crash-4.0-4.8/extensions/sial.c 2007-11-12 21:19:03.000000000 -0800 @@ -638,7 +638,20 @@ run_callback(void) { extern char *crash_global_cmd(); +extern FILE *crash_cmd_filep(); + + FILE *fp, + *ofp = NULL; + + if ((fp = crash_cmd_filep()) != NULL) { + ofp = sial_getofile(); + sial_setofile(fp); + } + sial_cmd(crash_global_cmd(), args, argcnt); + if (ofp) { + sial_setofile(ofp); + } } Index: crash-4.0-4.8/extensions.c =================================================================== --- crash-4.0-4.8-orig/extensions.c 2007-11-12 21:15:36.000000000 -0800 +++ crash-4.0-4.8/extensions.c 2007-11-12 21:15:57.000000000 -0800 @@ -374,3 +374,15 @@ { return pc->cmd_table; } + +FILE * +crash_cmd_filep(void) +{ + if (pc->redirect & REDIRECT_TO_FILE) { + return pc->ofile; + } else if (pc->redirect & REDIRECT_TO_PIPE) { + return pc->pipe; + } else { + return NULL; + } +} ________________________________ From: crash-utility-bounces(a)redhat.com [mailto:crash-utility-bounces@redhat.com] On Behalf Of Chouinard, Luc Sent: Monday, November 12, 2007 5:20 PM To: Discussion list for crash utility usage, maintenance and development; crash-utility(a)redhat.com Subject: RE: [Crash-utility] sial output redirection Very useful thing to have. Looks good. Dave can you queue it? Luc -----Original Message----- From: crash-utility-bounces(a)redhat.com on behalf of Denman, Robert Sent: Mon 11/12/2007 6:43 PM To: crash-utility(a)redhat.com Subject: [Crash-utility] sial output redirection Luc, The following patch will allow the output of SIAL commands to be redirected to a file or pipe. I did not handle all cases supported by crash (e.g. multi-pipes), only simple output redirection. Would you please consider this change or something like it in the next release? Thanks. - Robert - Generated By: Teradata Corporation (Mon Nov 12 15:29:56 PST 2007) extensions.c | 12 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ extensions/sial.c | 7 +++++++++++++++++++++++++++++++ 2 files changed, 19 insertions(+), 0 deletions(-) Index: crash-4.0-4.8/extensions/sial.c =================================================================== --- crash-4.0-4.8-orig/extensions/sial.c 2007-10-30 08:51:54.000000000 -0 700 +++ crash-4.0-4.8/extensions/sial.c 2007-11-12 15:28:55.000000000 -0800 @@ -638,6 +638,13 @@ run_callback(void) { extern char *crash_global_cmd(); +extern FILE *crash_cmd_filep(); + + FILE *fp; + + if ((fp = crash_cmd_filep()) != NULL) { + sial_setofile(fp); + } sial_cmd(crash_global_cmd(), args, argcnt); } Index: crash-4.0-4.8/extensions.c =================================================================== --- crash-4.0-4.8-orig/extensions.c 2007-10-30 08:51:54.000000000 -0700 +++ crash-4.0-4.8/extensions.c 2007-11-12 15:27:21.000000000 -0800 @@ -374,3 +374,15 @@ { return pc->cmd_table; } + +FILE * +crash_cmd_filep(void) +{ + if (pc->redirect & REDIRECT_TO_FILE) { + return pc->ofile; + } else if (pc->redirect & REDIRECT_TO_PIPE) { + return pc->pipe; + } else { + return NULL; + } +} Confidentiality Notice This e-mail (including any attachments) is intended only for the recipients named above. It may contain confidential or privileged information and should not be read, copied or otherwise used by any other person. If you are not a named recipient, please notify the sender of that fact and delete the e-mail from your system.

Thanks Dave. You're absolutely right. I tested using the global fp and the redirections work. The only change needed is to sial.c. Generated By: Teradata Corporation (Tue Nov 13 06:50:50 PST 2007) extensions/sial.c | 11 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 11 insertions(+), 0 deletions(-) Index: crash-4.0-4.8/extensions/sial.c =================================================================== --- crash-4.0-4.8-orig/extensions/sial.c 2007-11-12 21:15:36.000000000 -0800 +++ crash-4.0-4.8/extensions/sial.c 2007-11-13 06:50:44.000000000 -0800 @@ -638,7 +638,18 @@ run_callback(void) { extern char *crash_global_cmd(); + + FILE *ofp = NULL; + + if (fp != NULL) { + ofp = sial_getofile(); + sial_setofile(fp); + } + sial_cmd(crash_global_cmd(), args, argcnt); + if (ofp) { + sial_setofile(ofp); + } } -----Original Message----- From: Dave Anderson [mailto:anderson@redhat.com] Sent: Tuesday, November 13, 2007 5:44 AM To: Discussion list for crash utility usage, maintenance and development Cc: Luc.Chouinard(a)trueposition.com; Denman, Robert Subject: Re: [Crash-utility] sial output redirection Chouinard, Luc wrote: I don't see the need for crash_cmd_filep()? Since sial.c has the global "fp" #extern'd, it should just be able to call sial_setofile(fp), and then all redirection types should be covered. Dave