[NaCl SDK] nacl_io: Add chmod/fchmod

native_client_sdk/src/libraries/nacl_io/dir_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/dir_node.h"
 
 #include <errno.h>
 #include <string.h>
 
 #include "nacl_io/log.h"
@@ skipping 51 matching lines @@
 
 Error DirNode::GetDents(size_t offs,
                         dirent* pdir,
                         size_t size,
                         int* out_bytes) {
   AUTO_LOCK(node_lock_);
   BuildCache_Locked();
   return cache_.GetDents(offs, pdir, size, out_bytes);
 }
 
+Error DirNode::Fchmod(mode_t mode) {
+  AUTO_LOCK(node_lock_);
+  SetMode(mode);
+  return 0;

[Sam Clegg, 2014/09/24 20:48:28]

Could we not just put this in the Nade base class?

[binji, 2014/09/24 21:30:50]

I thought about that. It made more sense to me to "opt-in" to supporting fchmod,
considering that some filesystems don't use the built-in stat_ at all. Having it
default to setting the node seemed weird.

+}
+
 Error DirNode::AddChild(const std::string& name, const ScopedNode& node) {
   AUTO_LOCK(node_lock_);
 
   if (name.empty()) {
     LOG_ERROR("Can't add child with no name.");
     return ENOENT;
   }
 
   if (name.length() >= MEMBER_SIZE(dirent, d_name)) {
     LOG_ERROR("Child name is too long: %" PRIuS " >= %" PRIuS,
@@ skipping 56 matching lines @@
 
   cache_built_ = true;
 }
 
 void DirNode::ClearCache_Locked() {
   cache_built_ = false;
   cache_.Reset();
 }
 
 }  // namespace nacl_io