[libvirt] [PATCH 07/15] virsh: Handle absence of SA_SIGINFO

Matthias Bolte matthias.bolte at googlemail.com
Thu Apr 8 18:20:07 UTC 2010


2010/4/7 Daniel P. Berrange <berrange at redhat.com>:
> On Mon, Mar 22, 2010 at 02:25:37AM +0100, Matthias Bolte wrote:
>> MinGW and gnulib don't provide SA_SIGINFO on Windows.
>> ---
>>  tools/virsh.c |    4 ++++
>>  1 files changed, 4 insertions(+), 0 deletions(-)
>>
>> diff --git a/tools/virsh.c b/tools/virsh.c
>> index 32895b2..1c932bd 100644
>> --- a/tools/virsh.c
>> +++ b/tools/virsh.c
>> @@ -411,11 +411,13 @@ static int disconnected = 0; /* we may have been disconnected */
>>   * We get here when a SIGPIPE is being raised, we can't do much in the
>>   * handler, just save the fact it was raised
>>   */
>> +#ifdef SA_SIGINFO
>>  static void vshCatchDisconnect(int sig, siginfo_t * siginfo,
>>                                 void* context ATTRIBUTE_UNUSED) {
>>      if ((sig == SIGPIPE) || (siginfo->si_signo == SIGPIPE))
>>          disconnected++;
>>  }
>> +#endif
>>
>>  /*
>>   * vshSetupSignals:
>> @@ -425,6 +427,7 @@ static void vshCatchDisconnect(int sig, siginfo_t * siginfo,
>>   */
>>  static void
>>  vshSetupSignals(void) {
>> +#ifdef SA_SIGINFO
>>      struct sigaction sig_action;
>>
>>      sig_action.sa_sigaction = vshCatchDisconnect;
>> @@ -432,6 +435,7 @@ vshSetupSignals(void) {
>>      sigemptyset(&sig_action.sa_mask);
>>
>>      sigaction(SIGPIPE, &sig_action, NULL);
>> +#endif
>>  }
>>
>>  /*
>> --
>
> THis patch doesn't appear to have been pushed...
>
>
> Regards,
> Daniel
>

You already included this into your "Fix Win32 portability problems"
patch. so this patch is not necessary anymore.

Matthias




More information about the libvir-list mailing list