]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
mds: use uint64_t for sum of exported inodes
authorPatrick Donnelly <pdonnell@redhat.com>
Mon, 17 Apr 2017 20:09:04 +0000 (16:09 -0400)
committerPatrick Donnelly <pdonnell@redhat.com>
Tue, 18 Apr 2017 02:56:51 +0000 (22:56 -0400)
An int is too likely to overflow.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
src/mds/Migrator.cc
src/mds/Migrator.h

index 65b698118b3fad1f8ceeb47a7c6c6d829f33a151..f375c086ff09bbf4ecae2f893c2b75f7d5ab1140 100644 (file)
@@ -1269,7 +1269,7 @@ void Migrator::export_go_synced(CDir *dir, uint64_t tid)
   // fill export message with cache data
   MExportDir *req = new MExportDir(dir->dirfrag(), it->second.tid);
   map<client_t,entity_inst_t> exported_client_map;
-  int num_exported_inodes = encode_export_dir(req->export_data,
+  uint64_t num_exported_inodes = encode_export_dir(req->export_data,
                                              dir,   // recur start point
                                              exported_client_map,
                                              now);
@@ -1430,12 +1430,12 @@ void Migrator::finish_export_inode(CInode *in, utime_t now, mds_rank_t peer,
 
 }
 
-int Migrator::encode_export_dir(bufferlist& exportbl,
+uint64_t Migrator::encode_export_dir(bufferlist& exportbl,
                                CDir *dir,
                                map<client_t,entity_inst_t>& exported_client_map,
                                utime_t now)
 {
-  int num_exported = 0;
+  uint64_t num_exported = 0;
 
   dout(7) << "encode_export_dir " << *dir << " " << dir->get_num_head_items() << " head items" << dendl;
   
index 241faf0a64d6dafe6b7b22461ffc0625ffa8dea7..3e1df9d43454ccd67fc80ce3c5139cc6a982aa38 100644 (file)
@@ -309,7 +309,7 @@ public:
                                map<client_t,Capability::Import>& peer_imported);
 
 
-  int encode_export_dir(bufferlist& exportbl,
+  uint64_t encode_export_dir(bufferlist& exportbl,
                        CDir *dir,
                        map<client_t,entity_inst_t>& exported_client_map,
                        utime_t now);