Make the default directory size returned by stat to be 4096

native_client_sdk/src/libraries/nacl_io/html5fs/html5_fs_node.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "nacl_io/html5fs/html5_fs_node.h"
 
 #include <errno.h>
 #include <fcntl.h>
 #include <ppapi/c/pp_completion_callback.h>
 #include <ppapi/c/pp_directory_entry.h>
 #include <ppapi/c/pp_errors.h>
 #include <ppapi/c/pp_file_info.h>
 #include <ppapi/c/ppb_file_io.h>
 #include <string.h>
 #include <vector>
 
 #include "nacl_io/filesystem.h"
 #include "nacl_io/getdents_helper.h"
 #include "nacl_io/html5fs/html5_fs.h"
 #include "nacl_io/kernel_handle.h"
 #include "nacl_io/osdirent.h"
 #include "nacl_io/pepper_interface.h"
 #include "sdk_util/auto_lock.h"
 
 namespace nacl_io {
 
 namespace {
 
+const int kEmptyDirSize = 4096;
+
 struct OutputBuffer {
   void* data;
   int element_count;
 };
 
 void* GetOutputBuffer(void* user_data, uint32_t count, uint32_t size) {
   OutputBuffer* output = static_cast<OutputBuffer*>(user_data);
   output->element_count = count;
   if (count) {
     output->data = malloc(count * size);
@@ skipping 110 matching lines @@
 
   PP_FileInfo info;
   int32_t result =
       file_ref_iface_->Query(fileref_resource_, &info, PP_BlockUntilComplete());
   if (result != PP_OK)
     return PPERROR_TO_ERRNO(result);
 
   // Fill in known info here.
   memcpy(stat, &stat_, sizeof(stat_));
 
+  stat->st_size = static_cast<off_t>(info.size);
+
   // Fill in the additional info from ppapi.
   switch (info.type) {
     case PP_FILETYPE_REGULAR:
       stat->st_mode |= S_IFREG;
       break;
     case PP_FILETYPE_DIRECTORY:
       stat->st_mode |= S_IFDIR;
+      // Hack the directory size
+      // In Linux, even a empty directory has size 4096
+      if (info.size == 0)

[Sam Clegg, 2015/08/13 22:29:01]

So I guess you can drop this conditional.  And add a comment that info.size is
always zero for directories.

[zhitingzhu, 2015/08/13 22:32:33]

Done.

+        stat->st_size = kEmptyDirSize;
       break;
     case PP_FILETYPE_OTHER:
     default:
       break;
   }
-  stat->st_size = static_cast<off_t>(info.size);
   stat->st_atime = info.last_access_time;
   stat->st_mtime = info.last_modified_time;
   stat->st_ctime = info.creation_time;
 
   return 0;
 }
 
 Error Html5FsNode::Read(const HandleAttr& attr,
                         void* buf,
                         size_t count,
@@ skipping 134 matching lines @@
   fileref_resource_ = 0;
   Node::Destroy();
 }
 
 Error Html5FsNode::Fchmod(mode_t mode) {
   // html5fs does not support any kinds of permissions or mode bits.
   return 0;
 }
 
 }  // namespace nacl_io