From 5bc77788ece229aac83b0605b6652f1e56db3d5c Mon Sep 17 00:00:00 2001
From: Natalia Portillo <claunia@claunia.com>
Date: Sun, 19 Sep 2021 21:16:40 +0100
Subject: [PATCH] Refactor IMediaImage.ReadSector(s) to return error status
 instead of buffer.

---
 Enums/ErrorNumber.cs      | 3 ++-
 Interfaces/IMediaImage.cs | 6 ++++--
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/Enums/ErrorNumber.cs b/Enums/ErrorNumber.cs
index fa0dfebd6..12e9ef704 100644
--- a/Enums/ErrorNumber.cs
+++ b/Enums/ErrorNumber.cs
@@ -170,6 +170,7 @@ namespace Aaru.CommonTypes.Enums
         /// <summary>Cannot open device</summary>
         CannotOpenDevice = 27,
         /// <summary>Cannot remove the existing database</summary>
-        CannotRemoveDatabase = -28
+        CannotRemoveDatabase = 28,
+        SectorNotFound=29
     }
 }
\ No newline at end of file
diff --git a/Interfaces/IMediaImage.cs b/Interfaces/IMediaImage.cs
index d05a3ba89..22b986d90 100644
--- a/Interfaces/IMediaImage.cs
+++ b/Interfaces/IMediaImage.cs
@@ -83,7 +83,8 @@ namespace Aaru.CommonTypes.Interfaces
         /// <summary>Reads a sector's user data.</summary>
         /// <returns>The sector's user data.</returns>
         /// <param name="sectorAddress">Sector address (LBA).</param>
-        byte[] ReadSector(ulong sectorAddress);
+        /// <param name="buffer"></param>
+        ErrorNumber ReadSector(ulong sectorAddress, out byte[] buffer);
 
         /// <summary>Reads a sector's tag.</summary>
         /// <returns>The sector's tag.</returns>
@@ -95,7 +96,8 @@ namespace Aaru.CommonTypes.Interfaces
         /// <returns>The sectors user data.</returns>
         /// <param name="sectorAddress">Starting sector address (LBA).</param>
         /// <param name="length">How many sectors to read.</param>
-        byte[] ReadSectors(ulong sectorAddress, uint length);
+        /// <param name="buffer"></param>
+        ErrorNumber ReadSectors(ulong sectorAddress, uint length, out byte[] buffer);
 
         /// <summary>Reads tag from several sectors.</summary>
         /// <returns>The sectors tag.</returns>