DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 0A6AD3858D26
Message-ID: <56737cf9-4336-474d-8e83-285e93f87988@maxrnd.com>
Date: Thu, 12 Sep 2024 22:42:46 -0700
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: pthread_sigqueue(): First parameter is incompatible with glibc
To: cygwin@cygwin.com
References: <2e7c69cd-2125-2e88-cc44-1fa63533cdcc@t-online.de>
Content-Language: en-US
In-Reply-To: <2e7c69cd-2125-2e88-cc44-1fa63533cdcc@t-online.de>
Precedence: list
From: Mark Geisert via Cygwin <cygwin@cygwin.com>
Reply-To: Mark Geisert <mark@maxrnd.com>
Content-Type: text/plain; charset="utf-8"; Format="flowed"
Errors-To: cygwin-bounces~archive-cygwin=delorie.com@cygwin.com
Sender: "Cygwin" <cygwin-bounces~archive-cygwin=delorie.com@cygwin.com>
Content-Transfer-Encoding: 8bit

Hi Christian,

On 9/12/2024 9:21 AM, Christian Franke via Cygwin wrote:
> Cygwin /usr/include/pthread.h:
> 
> int pthread_sigqueue (pthread_t *, int, const union sigval);
> 
> Linux /usr/include/x86_64-linux-gnu/bits/sigthread.h:
> 
> extern int pthread_sigqueue (pthread_t __threadid, int __signo,
>                               const union sigval __value) __THROW;
> 
> 
> It is apparently the case since the very first related patch:
> https://sourceware.org/pipermail/cygwin-patches/2012q1/007565.html
> 
> RCS file: /cvs/src/src/winsup/cygwin/thread.cc,v
> ...
> +extern "C" int
> +pthread_sigqueue (pthread_t *thread, int sig, const union sigval value)
> +{

Thanks for the report. There's a couple things wrong that I can see 
right off the bat. The first param should indeed be "pthread_t", and the 
Cygwin code that implements the function should treat that first param 
as a thread id. Currently it's treated as a 'pthread_t *' to match the 
prototype.

I'll work on a patch for these issues.

..mark


-- 
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple