DragonFly bugs List (threaded) for 2008-05
DragonFly BSD
DragonFly bugs List (threaded) for 2008-05
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

Re: Qt 4.4 QtConcurrent and libthread_xu


From: Aggelos Economopoulos <aoiko@xxxxxxxxxxxxxx>
Date: Thu, 8 May 2008 00:32:00 +0300

On Wednesday 07 May 2008, Johannes Hofmann wrote:
> Aggelos Economopoulos <aoiko@cc.ece.ntua.gr> wrote:
> > On Wednesday 07 May 2008, Matthew Dillon wrote:
> > [...]
> >>     In libthread_xu it assumes non-NULL and will crash.
> >> 
> >>     Try this patch.  It will do the same check that libc_r does.  I'm
> >>     not convinced that Qt isn't broken, though, Qt shouldn't be passing
> >>     NULL to the mutex functions, it should be passing the address of
> >>     a pthread_mutex_t which itself can be NULL, but it should be passing
> >>     NULL.
> > 
> > Still, it's bad form for a library function to crash just because somebody
> > called it with an obviously invalid argument.
> 
> Not sure about that. There are many illeagal pointers that can't be catched
> so why treat NULL differently?

Because NULL is guaranteed not to point to an object.

> And a crash with an obvious stack trace shows that there is a problem that
> should be fixed.

While an error of EINVAL is hard to debug?

Aggelos



[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]