| Message ID | CAN04cT65Dd0q-VcKgmr56JrZkt9dKv3Qkv-1aKHtgUydrV10ow@mail.gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Applied, thanks! BTW the patch whitespace was mangled; I had to tweak it to make it apply. Gmail is a common culprit here. sage On Tue, 27 Aug 2013, Roald van Loon wrote: > Trivial cleanup. There were still 3 references to g_conf in CephxKeyServer. > Replaced them in favor of cct->_conf. > > Signed-off-by: Roald J. van Loon <roaldvanloon@gmail.com> > --- > diff --git a/src/auth/cephx/CephxKeyServer.cc b/src/auth/cephx/CephxKeyServer.cc > index e0c8174..e57b557 100644 > --- a/src/auth/cephx/CephxKeyServer.cc > +++ b/src/auth/cephx/CephxKeyServer.cc > @@ -163,7 +163,7 @@ bool KeyServer::_check_rotating_secrets() > ldout(cct, 10) << __func__ << " added " << added << dendl; > data.rotating_ver++; > //data.next_rotating_time = ceph_clock_now(cct); > - //data.next_rotating_time += MIN(g_conf->auth_mon_ticket_ttl, > g_conf->auth_service_ticket_ttl); > + //data.next_rotating_time += MIN(cct->_conf->auth_mon_ticket_ttl, > cct->_conf->auth_service_ticket_ttl); > _dump_rotating_secrets(); > return true; > } > @@ -191,7 +191,7 @@ int KeyServer::_rotate_secret(uint32_t service_id) > RotatingSecrets& r = data.rotating_secrets[service_id]; > int added = 0; > utime_t now = ceph_clock_now(cct); > - double ttl = service_id == CEPH_ENTITY_TYPE_AUTH ? > g_conf->auth_mon_ticket_ttl : g_conf->auth_service_ticket_ttl; > + double ttl = service_id == CEPH_ENTITY_TYPE_AUTH ? > cct->_conf->auth_mon_ticket_ttl : cct->_conf->auth_service_ticket_ttl; > > while (r.need_new_secrets(now)) { > ExpiringCryptoKey ek; > @@ -424,7 +424,7 @@ int KeyServer::_build_session_auth_info(uint32_t > service_id, CephXServiceTicketI > { > info.service_id = service_id; > info.ticket = auth_ticket_info.ticket; > - info.ticket.init_timestamps(ceph_clock_now(cct), > g_conf->auth_service_ticket_ttl); > + info.ticket.init_timestamps(ceph_clock_now(cct), > cct->_conf->auth_service_ticket_ttl); > > generate_secret(info.session_key); > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/src/auth/cephx/CephxKeyServer.cc b/src/auth/cephx/CephxKeyServer.cc index e0c8174..e57b557 100644 --- a/src/auth/cephx/CephxKeyServer.cc +++ b/src/auth/cephx/CephxKeyServer.cc @@ -163,7 +163,7 @@ bool KeyServer::_check_rotating_secrets() ldout(cct, 10) << __func__ << " added " << added << dendl; data.rotating_ver++; //data.next_rotating_time = ceph_clock_now(cct); - //data.next_rotating_time += MIN(g_conf->auth_mon_ticket_ttl, g_conf->auth_service_ticket_ttl); + //data.next_rotating_time += MIN(cct->_conf->auth_mon_ticket_ttl, cct->_conf->auth_service_ticket_ttl); _dump_rotating_secrets();
Trivial cleanup. There were still 3 references to g_conf in CephxKeyServer. Replaced them in favor of cct->_conf. Signed-off-by: Roald J. van Loon <roaldvanloon@gmail.com> --- return true; } @@ -191,7 +191,7 @@ int KeyServer::_rotate_secret(uint32_t service_id) RotatingSecrets& r = data.rotating_secrets[service_id]; int added = 0; utime_t now = ceph_clock_now(cct); - double ttl = service_id == CEPH_ENTITY_TYPE_AUTH ? g_conf->auth_mon_ticket_ttl : g_conf->auth_service_ticket_ttl; + double ttl = service_id == CEPH_ENTITY_TYPE_AUTH ? cct->_conf->auth_mon_ticket_ttl : cct->_conf->auth_service_ticket_ttl; while (r.need_new_secrets(now)) { ExpiringCryptoKey ek; @@ -424,7 +424,7 @@ int KeyServer::_build_session_auth_info(uint32_t service_id, CephXServiceTicketI { info.service_id = service_id; info.ticket = auth_ticket_info.ticket; - info.ticket.init_timestamps(ceph_clock_now(cct), g_conf->auth_service_ticket_ttl); + info.ticket.init_timestamps(ceph_clock_now(cct), cct->_conf->auth_service_ticket_ttl); generate_secret(info.session_key); -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html