Change in osmo-bts[master]: lc15: fsync requires fd instead of file stream
Harald Welte has submitted this change and it was merged. ( https://gerrit.osmocom.org/9535 ) Change subject: lc15: fsync requires fd instead of file stream .. lc15: fsync requires fd instead of file stream Change-Id: I7efb7280363b3a021b30c3b4eacb3aab70e9c8da --- M src/osmo-bts-litecell15/misc/lc15bts_par.c 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/osmo-bts-litecell15/misc/lc15bts_par.c b/src/osmo-bts-litecell15/misc/lc15bts_par.c index 75314a4..af9d030 100644 --- a/src/osmo-bts-litecell15/misc/lc15bts_par.c +++ b/src/osmo-bts-litecell15/misc/lc15bts_par.c @@ -144,7 +144,7 @@ return -EIO; } - fsync(fp); + fsync(fileno(fp)); fclose(fp); return 0; } @@ -176,7 +176,7 @@ rc = fwrite(buf, 1, size, fp); -fsync(fp); +fsync(fileno(fp)); fclose(fp); return rc; @@ -225,7 +225,7 @@ fclose(fp); return -EIO; } - fsync(fp); + fsync(fileno(fp)); fclose(fp); return 0; -- To view, visit https://gerrit.osmocom.org/9535 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I7efb7280363b3a021b30c3b4eacb3aab70e9c8da Gerrit-Change-Number: 9535 Gerrit-PatchSet: 3 Gerrit-Owner: Pau Espin Pedrol Gerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder (102)
Change in osmo-bts[master]: lc15: fsync requires fd instead of file stream
Harald Welte has posted comments on this change. ( https://gerrit.osmocom.org/9535 ) Change subject: lc15: fsync requires fd instead of file stream .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/9535 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7efb7280363b3a021b30c3b4eacb3aab70e9c8da Gerrit-Change-Number: 9535 Gerrit-PatchSet: 1 Gerrit-Owner: Pau Espin Pedrol Gerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-Comment-Date: Mon, 11 Jun 2018 12:10:55 + Gerrit-HasComments: No Gerrit-HasLabels: Yes
Change in osmo-bts[master]: lc15: fsync requires fd instead of file stream
Pau Espin Pedrol has uploaded this change for review. ( https://gerrit.osmocom.org/9535 Change subject: lc15: fsync requires fd instead of file stream .. lc15: fsync requires fd instead of file stream Change-Id: I7efb7280363b3a021b30c3b4eacb3aab70e9c8da --- M src/osmo-bts-litecell15/misc/lc15bts_par.c 1 file changed, 3 insertions(+), 3 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/35/9535/1 diff --git a/src/osmo-bts-litecell15/misc/lc15bts_par.c b/src/osmo-bts-litecell15/misc/lc15bts_par.c index 75314a4..af9d030 100644 --- a/src/osmo-bts-litecell15/misc/lc15bts_par.c +++ b/src/osmo-bts-litecell15/misc/lc15bts_par.c @@ -144,7 +144,7 @@ return -EIO; } - fsync(fp); + fsync(fileno(fp)); fclose(fp); return 0; } @@ -176,7 +176,7 @@ rc = fwrite(buf, 1, size, fp); -fsync(fp); +fsync(fileno(fp)); fclose(fp); return rc; @@ -225,7 +225,7 @@ fclose(fp); return -EIO; } - fsync(fp); + fsync(fileno(fp)); fclose(fp); return 0; -- To view, visit https://gerrit.osmocom.org/9535 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I7efb7280363b3a021b30c3b4eacb3aab70e9c8da Gerrit-Change-Number: 9535 Gerrit-PatchSet: 1 Gerrit-Owner: Pau Espin Pedrol