fix nil fileEntry

[liver]

on Create, after addNewChild fe(fileEntry) is nil value

[hanwen]

needs a test that repros the problem.

[liver]

accessing a property of an object that does not exist.
For NodeWriter interface, in the case of FileHandle is nil.

[hanwen]

needs a test that demonstrates the crash.

[hanwen]

Your test should create a file system, mount it, and then trigger the crash. I suspect that creating a file in an FS that doesn't use file handles might do the trick.

[liver]

Changed the test.
It seems to me that in this case, the test is superfluous.
I have not changed or added any functionality or logic.
Added a check for the existence of an object before accessing its property.

[hanwen]

why are you proposing this fix then?

[liver]

because I get an error in the latest version

[liver]

fix nil pointer dereference
check for existence of object

[liver]

Is there any chance of correction?

if fe != nil {
    out.Fh = uint64(fe.fh)
}

[hanwen]

I am still waiting for a test that demonstrates a crash in a filesystem without this fix. If you have a FS that doesn't work, it should be straightforward to distill that into a test.

[hanwen]

https://review.gerrithub.io/c/hanwen/go-fuse/+/1204114

[hanwen]

release tagged as v2.6.4