free nvs->pass_rec on error in nouveau_shader_pass2(), nouveau_shader_2.c

in nouveau_shader_pass2()

210: struct pass2_rec *rec;
...
215: rec = calloc(1, sizeof(struct pass2_rec));
...
218: nvs->pass_rec = rec;

untested, please check.
--
also free nvs->pass_rec on error

---

index cd27dac..0bbcb8a 100644
@@ -229,6 +229,8 @@ nouveau_shader_pass2(nvsPtr nvs)
 			     ((nvsSubroutine*)nvs->program_tree)->insn_head)) {
 		free(nvs->program);
 		nvs->program = NULL;
+		free(nvs->pass_rec);
+		nvs->pass_rec = NULL;
 		return GL_FALSE;
 	}

Hi Roel,

All your patches correct valid flaws. However, as you might have noticed, the old DRI code doesn't see much development at the moment. The reason is that we've decided to jump in the gallium3D wagon before it was even finished. So you have to know that this piece of code is currently not maintained, and that you'd have to finish it by yourself mostly.

I can of course push your patches if you feel like actually working on finishing this old-scheme DRI driver. Just tell me.

Stephane

Thanks for your information. I'll focus on gallium3D development instead. However, since my patches correct valid flaws, maybe it's best to push these patches. Is it a lot of work? If not, please do.

Chances are that someone in the future copy/pastes code, and with that these flaws, or that at a later time it is decided that the original driver was the better.

Roel Kluin