Bug 12672 – fsync() does not work correctly, leaves files full of \0

Bug 12672 - fsync() does not work correctly, leaves files full of \0

Summary:

fsync() does not work correctly, leaves files full of \0

Status:	RESOLVED FIXED

Product:	SB Radio
Classification:	Unclassified
Component:	OS/Bootloader
Version:	Include FW version in comment
Platform:	PC Windows XP

Importance:	P1 normal (vote)
Target Milestone:	MP
Assigned To:	Felix Mueller

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2009-07-03 15:35 UTC by Richard Titmuss
Modified:	2009-09-08 09:17 UTC (History)
CC List:	2 users (show)

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Richard Titmuss 2009-07-03 15:35:42 UTC

Comment 1 Richard Titmuss 2009-07-14 11:15:32 UTC

This is not needed for MP, a work around is in place using sync()

Comment 2 James Richardson 2009-07-16 12:41:40 UTC

QA: if you see this happen, do the following from Serial:

  - ls -l /etc/network/interfaces

see if the file is 0 length or has a length

Comment 3 Richard Titmuss 2009-07-17 05:34:16 UTC

Felix to check /etc/network/interfaces and /etc/wpa_supplicant.conf are written correctly.

Comment 4 Felix Mueller 2009-07-20 05:17:22 UTC

r6675 fixes /etc/network/interfaces

To Do: /etc/wpa_supplicant.conf

Comment 5 Caleb Crome 2009-07-20 18:23:44 UTC

Felix, r6657 breaks ethernet for me.  I checked 6673, and it works fine.  With 6675, I cannot connect to ethernet.

Comment 6 Caleb Crome 2009-07-20 18:57:50 UTC

Turns out both ethernet and wifi were broken for me.

I reverted r6675 back to r6673 to see if this was causing my network issues, and sure enough, reverting Networking.lua back to 6673 made networking work again.

Comment 7 Caleb Crome 2009-07-20 18:58:21 UTC

BTW, my reversion was in r6691.

Comment 8 Felix Mueller 2009-07-21 01:07:57 UTC

Thanks, Caleb, for sorting this out so quickly. Too many branches and different states of the same file. I forgot to add "local System = require("jive.System")" because it is already in the non mp branch. My mistake. Sorry about that.

It's fixed now. r6697

Comment 9 Felix Mueller 2009-07-21 01:09:03 UTC

moving target etc. back

Comment 10 Felix Mueller 2009-07-22 07:24:53 UTC

wpa_supplicant.conf writing fixed in r6744