[NaCl SDK] nacl_io: Replace allocated ino with hash of path.

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 {
 
 struct OutputBuffer {
@@ skipping 89 matching lines @@
 
     uint32_t file_name_length;
     const char* file_name =
         var_iface_->VarToUtf8(file_name_var, &file_name_length);
 
     if (file_name) {
       file_name_length =
           std::min(static_cast<size_t>(file_name_length),
                    MEMBER_SIZE(dirent, d_name) - 1);  // -1 for NULL.
 
-      // TODO(binji): Better handling of ino numbers.
-      helper.AddDirent(1, file_name, file_name_length);
+      // The INO is based on the running hash of fully qualified path, so
+      // a childs INO must be the parent directories hash, plus '/', plus
+      // the filename.
+      ino_t child_ino = Html5Fs::HashPathSegment(stat_.st_ino, file_name,
+                                                  file_name_length);

[binji, 2014/09/25 21:52:04]

nit: file_name_length should align with stat_ above.

[noelallen1, 2014/09/25 22:39:55]

Done.

+
+      helper.AddDirent(child_ino, file_name, file_name_length);
     }
 
     var_iface_->Release(file_name_var);
   }
 
   // Release the output buffer.
   free(output_buf.data);
 
   return helper.GetDents(offs, pdir, size, out_bytes);
 }
@@ skipping 165 matching lines @@
     filesystem_->ppapi()->ReleaseResource(fileio_resource_);
   }
 
   filesystem_->ppapi()->ReleaseResource(fileref_resource_);
   fileio_resource_ = 0;
   fileref_resource_ = 0;
   Node::Destroy();
 }
 
 }  // namespace nacl_io