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.

Sign up for GitHub

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

Jump to bottom

socket_base.cpp:162 comparison error #54

Closed
hintjens opened this issue Aug 29, 2010 · 1 comment
Closed

socket_base.cpp:162 comparison error #54

hintjens opened this issue Aug 29, 2010 · 1 comment

Comments

@hintjens
Copy link
Member

In master, this code is wrong:

    //  IPC transport is not available on Windows and OpenVMS.
#if defined ZMQ_HAVE_WINDOWS || defined ZMQ_HAVE_OPENVMS
    if (protocol_ != "ipc") {
        //  Unknown protocol.
        errno = EPROTONOSUPPORT;
        return -1;
    }
#endif

The comparison should be '==' not '!='.

Reported by Kelly Brock.
@sustrik
Copy link
Member

sustrik commented Aug 30, 2010

Fixed in rfba90af.

drahosp pushed a commit to LuaDist/libzmq that referenced this issue Feb 13, 2014
@hintjens
Merge pull request zeromq#54 from hintjens/master
ed9c744 
Backported fixes for LIBZMQ-464
@LutzWeischerFujitsu LutzWeischerFujitsu mentioned this issue Mar 10, 2021
Open
benjdero pushed a commit to benjdero/libzmq that referenced this issue Feb 20, 2023
@hintjens
Merge pull request zeromq#54 from miniway/master
e723d8f 
Apply the latest changes at protocol
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sustrik @hintjens