cmd/policy: Close file descriptor if required

[jiuker]

Please ensure your pull request adheres to the following guidelines:

• For first time contributors, read Submitting a pull request
• All code is covered by unit and/or runtime tests where feasible.
• All commits contain a well written commit description including a title,
  description and a Fixes: #XXX line if the commit addresses a particular
  GitHub issue.
• If your commit description contains a Fixes: <commit-id> tag, then
  please add the commit author[s] as reviewer[s] to this issue.
• All commits are signed off. See the section Developer’s Certificate of Origin
• Provide a title or release-note blurb suitable for the release notes.
• Thanks for contributing!

close file fd

Fixes: None

cmd/policy: Close file descriptor if required

[squeed]

Two things: can you properly set the labels?

And, secondly, rather than doing a type assertion, I'd suggest something like this:

fp, err := os.Open(path)
if err != nil {
	return nil, err
}
defer fp.Close()
r = fp

[jiuker]

Two things: can you properly set the labels?

And, secondly, rather than doing a type assertion, I'd suggest something like this:

fp, err := os.Open(path)
if err != nil {
	return nil, err
}
defer fp.Close()
r = fp

1.I don't have permission to modify pr's label
2.I have modified this pr like you requested
@squeed

[squeed]

/test

[squeed]

1.I don't have permission to modify pr's label

My mistake; added.

Tests seem to have failed for an unrelated reason; let's see if they go green.

[sayboras]

/test

[sayboras]

Conformance GatewayAPI failure is due to lack of rebase with master. Reviews are in, all CI jobs are good.

Marking this ready to merge.