test: fix pmem2_mover/TEST1 with memcheck for #5678

[grom72]

Fixes: #5678.

---

This change is 

[codecov]

Codecov Report

Merging #5679 (179b913) into stable-1.13 (67ce085) will decrease coverage by 2.02%.
The diff coverage is n/a.

❗ Current head 179b913 differs from pull request most recent head c727221. Consider uploading reports for the commit c727221 to get more accurate results

@@               Coverage Diff               @@
##           stable-1.13    #5679      +/-   ##
===============================================
- Coverage        74.27%   72.25%   -2.02%     
===============================================
  Files              145      145              
  Lines            22131    22633     +502     
  Branches          3704     3771      +67     
===============================================
- Hits             16437    16354      -83     
- Misses            5694     6279     +585     

[osalyk]

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @janekmi)

[janekmi]

Reviewed all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @grom72)

---

src/test/pmem2_mover/pmem2_mover.c line 120 at r1 (raw file):

	pmem2_source_delete(&src);
	CLOSE(fd);
	free(buffer_data);

Nitpick: It would be nice to create and destroy resources in precisely the opposite order. I know buffer_data is unrelated but still I consider it a good practice.

[grom72]

Reviewed 1 of 1 files at r2.
Reviewable status: all files reviewed (commit messages unreviewed), all discussions resolved (waiting on @grom72)

[janekmi]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @grom72)