async sid2gid

[simo]

Found this while working on the idmap code:

In nsswitch/winbindd_async.c
  in winbindd_dual_sid2gid()
    line 406

There seem to be a very odd call back to sid_to_gid() that I think
shouldn't be there (it could even loop if winbindd wasn't dual daemon as
sid_to_gid() calls back winbindd_sid_to_uid()).

It is probably completely safe to remove it, but I'd like to be 100%
sure someone didn't left it there for a reason.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer
email: idra at samba.org
http://samba.org