Commits
- Commit:
865dd3238a329a69864b4b9686567aa056d91c66
- From:
- Omar Polo <op@omarpolo.com>
- Date:
client_send_listener: send correct peerid
fix typo in previous commit where `0' was used instead of peerid.
Spotted by cage, thanks!
- Commit:
5c69b561db9f5713670e2f1e1519cc7f61b06645
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add IMSG_CTL_DEBUG to debug the opened fid
meant to be used from kamictl to dump all the opened fid by every
connection.
- Commit:
982f38dc950a803ee7331eba9f6b6f7a37d45f58
- From:
- Omar Polo <op@omarpolo.com>
- Date:
listener: handle IMSG_CTL_LOG_VERBOSE
adjust the logging level and forward it to every client process so they
can do so too.
- Commit:
9029ac6f3c1fa3e42c6fa231f8a45ed05d68b61e
- From:
- Omar Polo <op@omarpolo.com>
- Date:
wstat: missing read for size
9p stat message have a leading 2 bytes long size field. It's meant to
simplify the parsing, because while reading the contents of a
directory it's necessary to know how much long an entry is, so the
length field is present even in the Rstat reply and in the Twstat
input.
Previously we didn't consume that bit and thus mis-read all of the
following fields.
- Commit:
6d3d71a91bb82be60ffc572865e4329c40215dfe
- From:
- Omar Polo <op@omarpolo.com>
- Date:
kamid: simplify "not a directory" check
- Commit:
f4ad0146e973695127d1c9e1769cd2a7882184d8
- From:
- Omar Polo <op@omarpolo.com>
- Date:
work around wstat limitation
the previous commit added an explicit check for extra data in
messages. The problem is that wstat voluntarily skips reading some
fields at the end, so the len will always be greater than zero!
- Commit:
bcdb46a77ed7b4ff4d9e7ca8fae79fec74b504ae
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fail if the received packet is too long
This adds extra checks to every t* function: if the client sent too
much data per message, treat is an a protocol violation and terminate
the connection. Regress suite still passes, this is just an extra
safety check for finding wrong clients implementations.
The twrite function was the only one with a check like this.
- Commit:
ae6f257a8d22174cb2326cd60d690cceea59fa8e
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fix np_read{8,16,32,64} functions
they should return 0 on failure, not -1
- Commit:
46e4e74b0a512a6d37edcb8c711bc972f0adc908
- From:
- Omar Polo <op@omarpolo.com>
- Date:
kamid: change fid fname size to NAME_MAX
the fname field is just the file name, so NAME_MAX is more
appropriate.
Committing this separately from the rename change so it's easier to
roll back eventually.
- Commit:
543fc013a4eb973fc1249d5c6d06a0f27903399a
- From:
- Omar Polo <op@omarpolo.com>
- Date:
kamid: rename field fpath to fname
it's actually just the file name, not a path
- Commit:
755d75bfd052cfd45d0dc40872aa6454ecaff76f
- From:
- Omar Polo <op@omarpolo.com>
- Date:
kamid: add wstat
it's not exactly compliant, as 9p requires the changes to be done
atomically, but it's not possible to better targeting a UNIX file
system.
The 9p manpage regarding wstat isn't clear about the behaviour when
changing the name of the file: kamid changes the associated fid to
reflect the rename. kamid also allows to rename files to other
directories, the fid is updated to the new location even in this case.
- Commit:
68a7ef689646791cb998ff37308ac73f8ac9c2a5
- From:
- Omar Polo <op@omarpolo.com>
- Date:
the reported length for directories must be zero
plan9' stat(3) says so...
- Commit:
1e84c7b793afb2f25b7515283340a1f730290c3c
- From:
- Omar Polo <op@omarpolo.com>
- Date:
simplify client proc bootstrap
Drop the IMSG_AUTH_DIR imsg type and stuff everything in a struct
kd_auth_proc that gets sent via IMSG_AUTH. Tweak the necessary bits to
compile. While here also start documenting what data we send over which
imsg.
As an added benefit, now we're sending fixed-size messages instead of
variable length strings.
- Commit:
bbcba3edad343a751fe9bebefddfc812a26cc057
- From:
- Omar Polo <op@omarpolo.com>
- Date:
Merge branch 'portable'
Trying to maintain two branches was impossible at this stage for me
alone, so I'll recognize my stupidity and merge the portable branch into
the main one. The repository will build using autotools.
- Commit:
8a2c07d715dea281c0887cb83cced2593c550c5f
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fix build on linux