testgetdests waits indefinitely

OpenPrinting / libcups

OpenPrinting CUPS Library Sources

https://openprinting.github.io/cups/cups3.html

Apache License 2.0

35 stars 17 forks source link

Closed zdohnal closed 1 year ago

zdohnal commented 1 year ago

Hi,

I've tried to run the test suite to verify my PR, it gets stuck on cupsGetDests() function - here is the strace.txt.

Is it expected to work without CUPS 3.0 Local module?

I've configured and compiled the project like this:

$ ./configure
$ make
$ make test

michaelrsweet commented 1 year ago

@zdohnal It should work with a CUPS 2.x server running, and should time out after 10 seconds. Investigating...

michaelrsweet commented 1 year ago

OK, so "testgetdests" loops indefinitely. Changing to do a single test by default.

michaelrsweet commented 1 year ago

So it looks like the change has exposed a DNS-SD bug. Investigating...

michaelrsweet commented 1 year ago

It was a threading bug which I seem to have fixed...

[master 1428bee0a] Fix threading bug in DNS-SD API (Issue #33)

Still getting a "bad file descriptor" error in the Linux CI system, but we are probably not finding the cupsd socket...