Some clang static analyzer fixes.

More in a moment.
This commit is contained in:
JD Horelick 2010-10-23 22:30:52 -04:00
parent 331285b319
commit 6c7315ec36
2 changed files with 3 additions and 3 deletions

View File

@ -283,7 +283,7 @@ struct LocalUser
struct PrivilegeSet *privset; /* privset... */ struct PrivilegeSet *privset; /* privset... */
struct rb_event_t *override_timeout_event; struct ev_entry *override_timeout_event;
}; };
struct PreClient struct PreClient

View File

@ -317,7 +317,7 @@ single_whois(struct Client *source_p, struct Client *target_p, int operspy)
if(IsOper(target_p)) if(IsOper(target_p))
{ {
if(md = user_metadata_find(target_p, "OPERSTRING")) if((md = user_metadata_find(target_p, "OPERSTRING")) != NULL)
sendto_one_numeric(source_p, 313, "%s :%s", target_p->name, md->value); sendto_one_numeric(source_p, 313, "%s :%s", target_p->name, md->value);
else else
sendto_one_numeric(source_p, RPL_WHOISOPERATOR, form_str(RPL_WHOISOPERATOR), sendto_one_numeric(source_p, RPL_WHOISOPERATOR, form_str(RPL_WHOISOPERATOR),
@ -325,7 +325,7 @@ single_whois(struct Client *source_p, struct Client *target_p, int operspy)
IsService(target_p) ? ConfigFileEntry.servicestring : IsService(target_p) ? ConfigFileEntry.servicestring :
(IsAdmin(target_p) ? GlobalSetOptions.adminstring : (IsAdmin(target_p) ? GlobalSetOptions.adminstring :
GlobalSetOptions.operstring)); GlobalSetOptions.operstring));
if(md = user_metadata_find(target_p, "SWHOIS")) if((md = user_metadata_find(target_p, "SWHOIS")) != NULL)
sendto_one_numeric(source_p, 320, "%s :%s", target_p->name, md->value); sendto_one_numeric(source_p, 320, "%s :%s", target_p->name, md->value);
} }