seoxi/magick-rust
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #102 from BeatButton/write-image-blob-null

Browse Source 
don't segfault if MagickGetImageBlob returns null
This commit is contained in:
Nathan Fiedler
2023-04-07 11:31:08 -07:00
committed by GitHub
parent 1cfbc83471 c8fdace590
commit 257243c96c
1 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/wand/magick.rs
View File

@ -831,6 +831,9 @@ impl MagickWand {
bindings::MagickSetImageFormat(self.wand, c_format.as_ptr()); bindings::MagickSetImageFormat(self.wand, c_format.as_ptr());
bindings::MagickGetImageBlob(self.wand, &mut length) bindings::MagickGetImageBlob(self.wand, &mut length)
}; };
if blob.is_null() {
Err(MagickError("failed to write image blob"))
} else {
let mut bytes = Vec::with_capacity(length as usize); let mut bytes = Vec::with_capacity(length as usize);
unsafe { unsafe {
bytes.set_len(length as usize); bytes.set_len(length as usize);
@ -839,6 +842,7 @@ impl MagickWand {
}; };
Ok(bytes) Ok(bytes)
} }
}
/// Write the images in the desired format to a new blob. /// Write the images in the desired format to a new blob.
/// ///