laumann/ani - Change FM2KHERTELOSEZRVS6JVE4TWW2S75USLHXQSOIOYH5IQ4JECQIXQC

hunk: unconditionally free optional contents

For file add/del/undel, there's an optional "contents" atom that should also be freed (if present). As we do not yet have a way to indicate whether or not it's present, let us unconditionally call atomfree() on it.

Created by laumann on July 18, 2023

FM2KHERTELOSEZRVS6JVE4TWW2S75USLHXQSOIOYH5IQ4JECQIXQC

Dependencies

In channels

main

In discussions

#8 Investigate and fix resource leaks

Change contents

Replacement in hunk.c at line 68 [3.1372]

B:BD[2.299] → [2.299:375]

			/* if (b->filedel.contents) */
			/* 	atomfree(&b->filedel.contents); */

[2.299]

[2.375]

			/* FIXME this is optional */
			atomfree(&b->filedel.contents);

Replacement in hunk.c at line 76 [3.1372]

B:BD[2.523] → [2.523:576]

			/* FIXME check for optional fileundel.contents */

[2.523]

[2.576]

			/* FIXME this is optional */
			atomfree(&b->fileundel.contents);

Replacement in hunk.c at line 85 [3.1372]

B:BD[2.763] → [2.763:839]

			/* if (b->fileadd.contents) */
			/* 	atomfree(&b->fileadd.contents); */

[2.763]

[2.839]

			/* FIXME this is optional */
			atomfree(&b->fileadd.contents);