Skip to content

Commit

Permalink
Don't set the oper flags on client (#443)
Browse files Browse the repository at this point in the history
Remove unused OPER_FLAGS logic in favor new priv system. It had rotted over time and was overlapping with PINGSENT
  • Loading branch information
glguy authored Jan 29, 2025
1 parent 4e89f66 commit 7289d45
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 5 deletions.
2 changes: 0 additions & 2 deletions include/s_newconf.h
Original file line number Diff line number Diff line change
Expand Up @@ -140,8 +140,6 @@ extern void cluster_generic(struct Client *, const char *, int cltype,
#define OPER_ENCRYPTED 0x00001
#define OPER_NEEDSSL 0x80000

#define OPER_FLAGS 0 /* no oper privs in Client.flags/oper_conf.flags currently */

#define IsOperConfEncrypted(x) ((x)->flags & OPER_ENCRYPTED)
#define IsOperConfNeedSSL(x) ((x)->flags & OPER_NEEDSSL)

Expand Down
3 changes: 0 additions & 3 deletions ircd/s_user.c
Original file line number Diff line number Diff line change
Expand Up @@ -1177,8 +1177,6 @@ user_mode(struct Client *client_p, struct Client *source_p, int parc, const char
if(MyConnect(source_p))
{
source_p->umodes &= ~ConfigFileEntry.oper_only_umodes;
source_p->flags &= ~OPER_FLAGS;

rb_dlinkFindDestroy(source_p, &local_oper_list);
}

Expand Down Expand Up @@ -1494,7 +1492,6 @@ oper_up(struct Client *source_p, struct oper_conf *oper_p)
SetExtendChans(source_p);
SetExemptKline(source_p);

source_p->flags |= oper_p->flags;
source_p->user->opername = rb_strdup(oper_p->name);
source_p->user->privset = privilegeset_ref(oper_p->privset);

Expand Down

0 comments on commit 7289d45

Please sign in to comment.