> On Fri, Mar 3, 2017 at 10:15 AM, Elena Reshetova > <elena.reshetova@xxxxxxxxx> wrote: > > Now when new refcount_t type and API are finally merged > > (see include/linux/refcount.h), the following > > patches convert various refcounters in the ceph filesystem from atomic_t > > to refcount_t. By doing this we prevent intentional or accidental > > underflows or overflows that can led to use-after-free vulnerabilities. > > > > The below patches are fully independent and can be cherry-picked separately. > > Since we convert all kernel subsystems in the same fashion, resulting > > in about 300 patches, we have to group them for sending at least in some > > fashion to be manageable. Please excuse the long cc list. > > > > Not runtime tested, since I am not sure how to do it for ceph. > > However changes are pretty trivial in this case. > > > > Elena Reshetova (2): > > fs, ceph: convert ceph_mds_session.s_ref from atomic_t to refcount_t > > fs, ceph: convert ceph_cap_snap.nref from atomic_t to refcount_t > > > > fs/ceph/caps.c | 4 ++-- > > fs/ceph/mds_client.c | 18 +++++++++--------- > > fs/ceph/mds_client.h | 5 +++-- > > fs/ceph/snap.c | 2 +- > > fs/ceph/super.h | 5 +++-- > > 5 files changed, 18 insertions(+), 16 deletions(-) > > I'll pull these into testing. Just a check: how did testing went on this? If everything is good, can you merge the changes? Best Regards, Elena. > > Thanks, > > Ilya ��.n��������+%������w��{.n����z��u���ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f