Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cupsaddsmb doesn't handle domain sockets #2556

Closed
michaelrsweet opened this issue Oct 11, 2007 · 1 comment
Closed

cupsaddsmb doesn't handle domain sockets #2556

michaelrsweet opened this issue Oct 11, 2007 · 1 comment
Milestone

Comments

@michaelrsweet
Copy link
Collaborator

Version: 1.3-current
CUPS.org User: pipitas

Sine CUPS 1.2, users tryin to install the Windows/CUPS/Samba drivers very frequently encounter the same problem as described by this forum post, where a user did run "cupsaddsmb -v -U root -a":

http://www.cups.org/newsgroups.php?gcups.general+v:30889

The message displayed fortunately shows where the problem lays:

"[....] smbclient ///var/run/cups/cups.sock/print$ [....]"

It happens if there is support for unix domain socket connections (default in 1.2.x++) pre-compiled into the CUPS package.

However, smbclient (called by cupsaddsmb) does not work over unix domain sockets, it requires a TCP socket.

The man page (as well as the little help message displayed by "cupsaddsmb --foobar") should clearly hint at the fact that since 1.2 the correct usage of "-H" is required.

I think it should even be made mandatory in order to avoid this problem. (Don't ever default to any "localhost" assumption either....)

@michaelrsweet
Copy link
Collaborator Author

CUPS.org User: mike

Fixed in Subversion repository.

The real issue is that while we default the Samba server to the same host as the CUPS server, we weren't checking for a domain socket address. Mapping domain sockets to localhost preserves the prior behavior...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant