pkg/fileutil: fix constant for linux locking #12440
Conversation
The constant F_OFD_GETLK is 36, not 37, according to /usr/include/bits/fcntl-linux.h Credits go to joakim-tjernlund who digged deep enough to find this. Fixes #31182
| @@ -30,7 +30,7 @@ import ( | |||
| // | |||
| // constants from /usr/include/bits/fcntl-linux.h | |||
| const ( | |||
| F_OFD_GETLK = 37 | |||
| F_OFD_GETLK = 36 | |||
There was a problem hiding this comment.
Perhaps it'll be better to reuse golang.org/x/sys/unix instead, which have stuff like this available.
There was a problem hiding this comment.
very well. I'm out for lack of qualification, sorry
There was a problem hiding this comment.
I suggest you pull this one to get rid of the problem, then add your cleanup on top.
There was a problem hiding this comment.
Agree. This can be merged as-is.
There was a problem hiding this comment.
Alternatively, I have created #12444 which I will rebase if this one will be merged earlier.
There was a problem hiding this comment.
Verified with:
https://code.woboq.org/gcc/include/bits/fcntl-linux.h.html#_M/F_OFD_GETLK
I would recommend adding a test that confirms proper behavior of the lock library.
|
This problem is still not fixed in version 19 branch which I guess most people use in production |
The constant F_OFD_GETLK is 36, not 37, according to
/usr/include/bits/fcntl-linux.h
Credits go to joakim-tjernlund who digged deep enough
to find this.
Fixes #31182