illumos gate

There is following piece of code in nb_send():

1658    if (!(ntohl(*(uint32_t *)buff) & 2^31)) {
1659        return (-1);
1660    }

These lines should be removed for various reasons:

1. The original author probably wanted to write "2 << 31" (shift) instead of "2^31" (xor),
2. according the operator precedences, "ntohl(...) & 2" is evaluated first. The "^31" is evaluated after "&",
3. based on the above, the condition at line 1658 is always FALSE,
4. there is no need for such check, because the beginning of the buff data have no special meaning. This seems to be an artifact from deep (and already forgotten) history.