Merge pull request #2499 from rtfeldman/ident-str-drop

Fix invalid memory access in IdentStr Drop impl
This commit is contained in:
Folkert de Vries 2022-02-15 23:04:29 +01:00 committed by GitHub
commit e2da2e86c2
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -311,7 +311,7 @@ impl Clone for IdentStr {
impl Drop for IdentStr { impl Drop for IdentStr {
fn drop(&mut self) { fn drop(&mut self) {
if !self.is_small_str() { if !self.is_empty() && !self.is_small_str() {
unsafe { unsafe {
let align = mem::align_of::<u8>(); let align = mem::align_of::<u8>();
let layout = Layout::from_size_align_unchecked(self.length, align); let layout = Layout::from_size_align_unchecked(self.length, align);