Close the file handle before exiting the function

Close the file handle before exiting the function

When exiting the function because cannot open "oom_adj" file, not close the file handle "dirfd".

BUG=n/a

Committed: https://crrev.com/35feaa96a87aad71ed508b825ebc85f9dacd0977
Cr-Commit-Position: refs/heads/master@{#427018}

[yan0422.wang, 2016-10-18 12:23:39]

Description was changed from

==========
Close the file handle before exiting the function

When exiting the function because cannot open "oom_adj" file, not close the file
handle "dirfd".

BUG=n/a
==========

to

==========
Close the file handle before exiting the function

When exiting the function because cannot open "oom_adj" file, not close the file
handle "dirfd".

BUG=n/a
==========

[yan0422.wang, 2016-10-18 12:23:39]

yan0422.wang@samsung.com changed reviewers:
+ jln@chromium.org, jorgelo@chromium.org

[yan0422.wang, 2016-10-18 12:28:16]

Fixed a  file handle not close issue.

line 61 in sandbox/linux/suid/process_util_linux.c

[Jorge Lucangeli Obes, 2016-10-18 13:38:52]

jorgelo@chromium.org changed reviewers:
+ rickyz@chromium.org

[Jorge Lucangeli Obes, 2016-10-18 13:38:52]

This looks reasonable to me, although the setuid code is mostly deprecated and
AFAICT only kept around for distros who don't want to use user namespaces, right
Ricky?

lgtm

[rickyz (no longer on Chrome), 2016-10-18 20:19:50]

lgtm

[yan0422.wang, 2016-10-19 02:37:46]

The CQ bit was checked by yan0422.wang@samsung.com

[commit-bot: I haz the power, 2016-10-19 02:38:34]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-19 02:38:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-19 02:38:37]

The author yan0422.wang@samsung.com has not signed Google Contributor License
Agreement. Please visit https://cla.developers.google.com to sign and manage
CLA.

[yan0422.wang, 2016-10-19 03:01:54]

The CQ bit was checked by yan0422.wang@samsung.com

[commit-bot: I haz the power, 2016-10-19 03:04:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-19 03:04:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-19 03:04:25]

The author yan0422.wang@samsung.com has not signed Google Contributor License
Agreement. Please visit https://cla.developers.google.com to sign and manage
CLA.

[Jorge Lucangeli Obes, 2016-10-19 13:35:02]

On 2016/10/19 03:04:25, commit-bot: I haz the power wrote:
> The author mailto:yan0422.wang@samsung.com has not signed Google Contributor
License
> Agreement. Please visit https://cla.developers.google.com to sign and manage
> CLA.

Wang, you're gonna have to sign the CLA before you can submit. Re-checking the
commit box is not gonna work.

[yan0422.wang, 2016-10-24 05:27:00]

The CQ bit was checked by yan0422.wang@samsung.com

[commit-bot: I haz the power, 2016-10-24 05:27:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-24 05:52:32]

Description was changed from

==========
Close the file handle before exiting the function

When exiting the function because cannot open "oom_adj" file, not close the file
handle "dirfd".

BUG=n/a
==========

to

==========
Close the file handle before exiting the function

When exiting the function because cannot open "oom_adj" file, not close the file
handle "dirfd".

BUG=n/a
==========

[commit-bot: I haz the power, 2016-10-24 05:52:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-24 05:54:32]

Description was changed from

==========
Close the file handle before exiting the function

When exiting the function because cannot open "oom_adj" file, not close the file
handle "dirfd".

BUG=n/a
==========

to

==========
Close the file handle before exiting the function

When exiting the function because cannot open "oom_adj" file, not close the file
handle "dirfd".

BUG=n/a

Committed: https://crrev.com/35feaa96a87aad71ed508b825ebc85f9dacd0977
Cr-Commit-Position: refs/heads/master@{#427018}
==========

[commit-bot: I haz the power, 2016-10-24 05:54:33]

Patchset 1 (id:??) landed as
https://crrev.com/35feaa96a87aad71ed508b825ebc85f9dacd0977
Cr-Commit-Position: refs/heads/master@{#427018}

[lgombos, 2016-10-25 02:21:59]

On 2016/10/19 13:35:02, Jorge Lucangeli Obes wrote:
> On 2016/10/19 03:04:25, commit-bot: I haz the power wrote:
> > The author mailto:yan0422.wang@samsung.com has not signed Google Contributor
> License
> > Agreement. Please visit https://cla.developers.google.com to sign and manage
> > CLA.
> 
> Wang, you're gonna have to sign the CLA before you can submit. Re-checking the
> commit box is not gonna work.

I added Wang to the CLA (just now). Sorry for the delay. 
Not sure why the patch landed, but Wang should be all set now.