CiviGroup Roles Sync does not trigger sync on Drupal user login

When a user has been added to a CiviCRM group using an API or smart group, then the next time they login to drupal (with their uf matched drupal account) their Drupal role should be updated per the CiviGroup Roles Sync module.

Although there is a bug in the code which prevents this:

civicrm_group_roles.module
85 $contactid = civicrm_api('uf_match', 'getvalue',array('version' => 3, 'uf_id' => $user->uid, 'return' => 'contact_id'));
86
87 if (empty($contact))

Line 87 should be:
87 if (empty($contactid)) {

There is no variable $contact in this function and this function will always return true and not finish.