diff options
| author | Andrew Tridgell <tridge@samba.org> | 2000-03-26 23:16:07 +0000 |
|---|---|---|
| committer | Andrew Tridgell <tridge@samba.org> | 2000-03-26 23:16:07 +0000 |
| commit | 94ca1d1250d12918311a402cc689050ba1d06e24 (patch) | |
| tree | fcba2835d78cc7656c821057e37afdba84ea8014 | |
| parent | b13a903720a6ef789ee7d688c05be2e77584580e (diff) | |
| download | samba-94ca1d1250d12918311a402cc689050ba1d06e24.tar.gz samba-94ca1d1250d12918311a402cc689050ba1d06e24.tar.xz samba-94ca1d1250d12918311a402cc689050ba1d06e24.zip | |
when doing a "secure nbns" wack response and check with owner for a
multihomed registration we were sending the packet to the wrong host!
We sent it to the person trying to register rather than to the person
who currently holds the name. That means we were not secure and we
could allow two people to register the same name.
Jeremy, you may wish to merge this change into 2.0.X
| -rw-r--r-- | source/nmbd/nmbd_winsserver.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/source/nmbd/nmbd_winsserver.c b/source/nmbd/nmbd_winsserver.c index e978718106a..2b99a8ce53b 100644 --- a/source/nmbd/nmbd_winsserver.c +++ b/source/nmbd/nmbd_winsserver.c @@ -1176,9 +1176,12 @@ is one of our (WINS server) names. Denying registration.\n", nmb_namestr(questio * the recursion desired flag is not set (you were right Luke :-). * This function should *only* be called from the WINS server * code. JRA. + * + * Note that this packet is sent to the current owner of the name, + * not the person who sent the packet */ - query_name_from_wins_server( p->ip, + query_name_from_wins_server( namerec->data.ip[0], question->name, question->name_type, wins_multihomed_register_query_success, |