fix fuzzing error in ReadContent (#214)

ydb-platform · Jan 24, 2024 · 73b30ae · 73b30ae
1 parent 5fb1854
commit 73b30ae
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/cloud/filestore/libs/vfs_fuse/fs_impl_list.cpp b/cloud/filestore/libs/vfs_fuse/fs_impl_list.cpp
@@ -67,7 +67,7 @@ class TDirectoryHandle
         return TDirectoryContent{content, 0, size};
     }
 
-    TMaybe<TDirectoryContent> ReadContent(size_t size, size_t offset)
+    TMaybe<TDirectoryContent> ReadContent(size_t size, size_t offset, TLog& Log)
     {
         size_t end = 0;
         TBufferPtr content = nullptr;
@@ -85,8 +85,10 @@ class TDirectoryHandle
         TDirectoryContent result;
         if (content) {
             offset = offset - (end - content->size());
-            Y_ABORT_UNLESS(offset < content->size(), "off %lu size %lu",
-                offset, content->size());
+            if (offset >= content->size()) {
+                STORAGE_ERROR("off %lu size %lu", offset, content->size());
+                return Nothing();
+            }
             result = {content, offset, size};
         }
 
@@ -277,7 +279,7 @@ void TFileSystem::ReadDir(
     if (!offset) {
         // directory contents need to be refreshed on rewinddir()
         handle->ResetContent();
-    } else if (auto content = handle->ReadContent(size, offset)) {
+    } else if (auto content = handle->ReadContent(size, offset, Log)) {
         reply(*this, *content);
         return;
     }