more error checking around images, should prevent vram leaks

flake.lock

@@ -21,17 +21,17 @@
     "imgui": {
       "flake": false,
       "locked": {
-        "lastModified": 1752080038,
+        "lastModified": 1758128722,
-        "narHash": "sha256-m5h/H8k/CZ5Qwr3L9+Elt8XT/Tgwr3s4HuhVtI4wkwE=",
+        "narHash": "sha256-J+h7jJ+4wqr6RivtzyTDMXKxFoGs7dQbzqdu51XgEbc=",
         "owner": "ocornut",
         "repo": "imgui",
-        "rev": "5d4126876bc10396d4c6511853ff10964414c776",
+        "rev": "bf75bfec48fc00f532af8926130b70c0e26eb099",
         "type": "github"
       },
       "original": {
         "owner": "ocornut",
         "repo": "imgui",
-        "rev": "5d4126876bc10396d4c6511853ff10964414c776",
+        "rev": "bf75bfec48fc00f532af8926130b70c0e26eb099",
         "type": "github"
       }
     },

flake.nix

@@ -21,7 +21,7 @@
       flake = false;
     };
     imgui = {
-      url = "github:ocornut/imgui/5d4126876bc10396d4c6511853ff10964414c776";
+      url = "github:ocornut/imgui/bf75bfec48fc00f532af8926130b70c0e26eb099";
       flake = false;
     };
     plutosvg = {

src/image_loader_webp.cpp

@@ -70,7 +70,10 @@ ImageLoaderWebP::ImageResult ImageLoaderWebP::loadFromMemoryRGBA(const uint8_t*
 	while (WebPAnimDecoderHasMoreFrames(dec.get())) {
 		uint8_t* buf;
 		int timestamp;
-		WebPAnimDecoderGetNext(dec.get(), &buf, &timestamp);
+		if (WebPAnimDecoderGetNext(dec.get(), &buf, &timestamp) == 0 || buf == nullptr) {
+			// error out with empty res
+			return {};
+		}
 		// ... (Render 'buf' based on 'timestamp').
 		// ... (Do NOT free 'buf', as it is owned by 'dec').
 

src/message_image_loader.cpp

@@ -103,10 +103,17 @@ TextureLoaderResult MessageImageLoader::load(TextureUploaderI& tu, Message3Handl
 		new_entry.current_texture = 0;
 		for (const auto& [ms, data] : res.frames) {
 			const auto n_t = tu.upload(data.data(), res.width, res.height);
+			if (n_t == 0) {
+				continue;
+			}
 			new_entry.textures.push_back(n_t);
 			new_entry.frame_duration.push_back(ms);
 		}
 
+		if (new_entry.textures.empty()) {
+			continue;
+		}
+
 		new_entry.width = res.width;
 		new_entry.height = res.height;
 

src/texture_cache.hpp

@@ -116,6 +116,9 @@ struct TextureCache {
 				_tu.destroy(tex_id);
 			}
 		}
+		for (const auto& tex_id : _default_texture.textures) {
+			_tu.destroy(tex_id);
+		}
 	}
 
 	struct GetInfo {