fix: improve frame wrapping logic and error handling in rendering code

- Updated error message in FS_OpenFileInPak to include a newline for better readability.
- Added checks for valid frame counts in animation and mesh rendering functions across OpenGL and Vulkan implementations to prevent division by zero errors.
- Refactored frame wrapping logic to use a local variable for clarity and performance.

Author: timfox

src/qcommon/files.c

@@ -1570,7 +1570,7 @@ static int FS_OpenFileInPak( fileHandle_t *file, pack_t *pak, fileInPack_t *pakF
 		// open a new file on the pakfile
 		temp = unzReOpen( pak->pakFilename, pak->handle );
 		if ( temp == NULL ) {
-			Com_Printf( S_COLOR_RED "Couldn't reopen %s", pak->pakFilename );
+			Com_Printf( S_COLOR_RED "Couldn't reopen %s\n", pak->pakFilename );
 			*file = FS_INVALID_HANDLE;
 			return -1;
 		}

src/renderers/opengl/tr_animation.c

@@ -191,10 +191,15 @@ void R_MDRAddAnimSurfaces( trRefEntity_t *ent ) {
 
 	personalModel = (ent->e.renderfx & RF_THIRD_PERSON) && (tr.viewParms.portalView == PV_NONE);
 
+	if ( !header || header->numFrames < 1 ) {
+		return;
+	}
+
 	if ( ent->e.renderfx & RF_WRAP_FRAMES )
 	{
-		ent->e.frame %= header->numFrames;
-		ent->e.oldframe %= header->numFrames;
+		const int nf = (int)header->numFrames;
+		ent->e.frame %= nf;
+		ent->e.oldframe %= nf;
 	}
 
 	//

src/renderers/opengl/tr_mesh.c

@@ -300,9 +300,14 @@ void R_AddMD3Surfaces( trRefEntity_t *ent ) {
 	// don't add third_person objects if not in a portal
 	personalModel = (ent->e.renderfx & RF_THIRD_PERSON) && (tr.viewParms.portalView == PV_NONE);
 
+	if ( !tr.currentModel->md3[0] || tr.currentModel->md3[0]->numFrames < 1 ) {
+		return;
+	}
+
 	if ( ent->e.renderfx & RF_WRAP_FRAMES ) {
-		ent->e.frame %= tr.currentModel->md3[0]->numFrames;
-		ent->e.oldframe %= tr.currentModel->md3[0]->numFrames;
+		const int nf = tr.currentModel->md3[0]->numFrames;
+		ent->e.frame %= nf;
+		ent->e.oldframe %= nf;
 	}
 
 	//

src/renderers/opengl/tr_model_iqm.c

@@ -1073,7 +1073,7 @@ void R_AddIQMSurfaces( trRefEntity_t *ent ) {
 	// don't add third_person objects if not in a portal
 	personalModel = (ent->e.renderfx & RF_THIRD_PERSON) && (tr.viewParms.portalView == PV_NONE);
 
-	if ( ent->e.renderfx & RF_WRAP_FRAMES ) {
+	if ( ent->e.renderfx & RF_WRAP_FRAMES && data->num_frames > 0 ) {
 		ent->e.frame %= data->num_frames;
 		ent->e.oldframe %= data->num_frames;
 	}

src/renderers/vulkan/shaders/spirv/generated/shader_build_meta.txt

@@ -1,5 +1,5 @@
 glslang_validator_path=/usr/bin/glslangValidator
 glslang_validator_version=Glslang Version: 11:16.2.0 
-generated_at=2026-04-27T03:13:41Z
+generated_at=2026-04-27T05:04:11Z
 shader_data_sha256=b6c7f805a5810cddfd7365a786824d42ea896f25f1d35c3ec412e157eaeee493
 shader_binding_sha256=960921d52493ab4fcf2f8b60edb34fa0b64a852c47b5ad2ae3abdf06c35ebe4a

src/renderers/vulkan/tr_animation.c

@@ -192,10 +192,15 @@ void R_MDRAddAnimSurfaces( trRefEntity_t *ent ) {
 
 	personalModel = (ent->e.renderfx & RF_THIRD_PERSON) && (tr.viewParms.portalView == PV_NONE);
 
+	if ( !header || header->numFrames < 1 ) {
+		return;
+	}
+
 	if ( ent->e.renderfx & RF_WRAP_FRAMES )
 	{
-		ent->e.frame %= header->numFrames;
-		ent->e.oldframe %= header->numFrames;
+		const int nf = (int)header->numFrames;
+		ent->e.frame %= nf;
+		ent->e.oldframe %= nf;
 	}
 
 	//

src/renderers/vulkan/tr_mesh.c

@@ -300,9 +300,14 @@ void R_AddMD3Surfaces( trRefEntity_t *ent ) {
 	// don't add third_person objects if not in a portal
 	personalModel = (ent->e.renderfx & RF_THIRD_PERSON) && (tr.viewParms.portalView == PV_NONE);
 
+	if ( !tr.currentModel->md3[0] || tr.currentModel->md3[0]->numFrames < 1 ) {
+		return;
+	}
+
 	if ( ent->e.renderfx & RF_WRAP_FRAMES ) {
-		ent->e.frame %= tr.currentModel->md3[0]->numFrames;
-		ent->e.oldframe %= tr.currentModel->md3[0]->numFrames;
+		const int nf = tr.currentModel->md3[0]->numFrames;
+		ent->e.frame %= nf;
+		ent->e.oldframe %= nf;
 	}
 
 	//

src/renderers/vulkan/tr_model_iqm.c

@@ -1562,7 +1562,7 @@ void R_AddIQMSurfaces( trRefEntity_t *ent ) {
 	// don't add third_person objects if not in a portal
 	personalModel = (ent->e.renderfx & RF_THIRD_PERSON) && (tr.viewParms.portalView == PV_NONE);
 
-	if ( ent->e.renderfx & RF_WRAP_FRAMES ) {
+	if ( ent->e.renderfx & RF_WRAP_FRAMES && data->num_frames > 0 ) {
 		ent->e.frame %= data->num_frames;
 		ent->e.oldframe %= data->num_frames;
 	}

src/renderers/vulkan/vk_instance.c

@@ -188,6 +188,8 @@ static qboolean vk_create_device( VkPhysicalDevice physical_device, int device_i
 	VkPhysicalDevice8BitStorageFeatures storage_8bit_features;
 #endif
 	VkPhysicalDeviceMeshShaderFeaturesNV mesh_shader_features_nv;
+	/* Must survive until qvkCreateDevice (do not declare inside a narrow if-block). */
+	VkPhysicalDeviceHostQueryResetFeatures host_query;
 
 	ri.Printf( PRINT_ALL, "...selected physical device: %i\n", device_index );
 
@@ -681,7 +683,6 @@ static qboolean vk_create_device( VkPhysicalDevice physical_device, int device_i
 		}
 #endif
 		if ( hostQueryReset ) {
-			VkPhysicalDeviceHostQueryResetFeatures host_query;
 			const void *prev_next = device_desc.pNext;
 			Com_Memset( &host_query, 0, sizeof( host_query ) );
 			host_query.sType = VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_HOST_QUERY_RESET_FEATURES;