]> git.ipfire.org Git - thirdparty/xz.git/commitdiff
xz: Fix a too relaxed assertion and remove uses of SSIZE_MAX.
authorLasse Collin <lasse.collin@tukaani.org>
Thu, 31 Aug 2023 15:14:43 +0000 (18:14 +0300)
committerLasse Collin <lasse.collin@tukaani.org>
Thu, 31 Aug 2023 15:14:43 +0000 (18:14 +0300)
SSIZE_MAX isn't readily available on MSVC. Removing it means
that there is one thing less to worry when porting to MSVC.

src/xz/file_io.c
src/xz/file_io.h

index a181b531258f8d5b3aff4c1d1d5e624f0c195dfc..767c819a18817176db965a50436c1063c776f684 100644 (file)
@@ -1157,8 +1157,7 @@ io_fix_src_pos(file_pair *pair, size_t rewind_size)
 extern size_t
 io_read(file_pair *pair, io_buf *buf, size_t size)
 {
-       // We use small buffers here.
-       assert(size < SSIZE_MAX);
+       assert(size <= IO_BUFFER_SIZE);
 
        size_t pos = 0;
 
@@ -1285,7 +1284,7 @@ is_sparse(const io_buf *buf)
 static bool
 io_write_buf(file_pair *pair, const uint8_t *buf, size_t size)
 {
-       assert(size < SSIZE_MAX);
+       assert(size <= IO_BUFFER_SIZE);
 
        while (size > 0) {
                const ssize_t amount = write(pair->dest_fd, buf, size);
index 8a9e33630db34f73fbe35857070c84202ec6b035..6992efa4f86cc9079165ba523c6d9200ed51223c 100644 (file)
@@ -118,7 +118,7 @@ extern void io_close(file_pair *pair, bool success);
 ///
 /// \param      pair    File pair having the source file open for reading
 /// \param      buf     Destination buffer to hold the read data
-/// \param      size    Size of the buffer; assumed be smaller than SSIZE_MAX
+/// \param      size    Size of the buffer; must be at most IO_BUFFER_SIZE
 ///
 /// \return     On success, number of bytes read is returned. On end of
 ///             file zero is returned and pair->src_eof set to true.
@@ -172,7 +172,7 @@ extern bool io_pread(file_pair *pair, io_buf *buf, size_t size, uint64_t pos);
 ///
 /// \param      pair    File pair having the destination file open for writing
 /// \param      buf     Buffer containing the data to be written
-/// \param      size    Size of the buffer; assumed be smaller than SSIZE_MAX
+/// \param      size    Size of the buffer; must be at most IO_BUFFER_SIZE
 ///
 /// \return     On success, zero is returned. On error, -1 is returned
 ///             and error message printed.