2003-01-30 Havoc Pennington <hp@pobox.com>
authorHavoc Pennington <hp@redhat.com>
Thu, 30 Jan 2003 04:20:44 +0000 (04:20 +0000)
committerHavoc Pennington <hp@redhat.com>
Thu, 30 Jan 2003 04:20:44 +0000 (04:20 +0000)
commit7ba714ad7fe8256edfaad7d9a0f09aeb9611ca44
tree921d4ee8a780d5fe03f168405a5811287c10a926
parent8fdd8915bd7424cdf90bf59a018838a1290ac0c4
2003-01-30  Havoc Pennington  <hp@pobox.com>

* dbus/dbus-message.c: use message->byte_order instead of
DBUS_COMPILER_BYTE_ORDER throughout.
(dbus_message_create_header): pad header to align the
start of the body of the message to 8-byte boundary

* dbus/dbus-marshal.h: make all the demarshalers take const
DBusString arguments.

* dbus/dbus-message.c (_dbus_message_loader_return_buffer):
validate message args here, so we don't have to do slow validation
later, and so we catch bad messages as they are incoming. Also add
better checks on header_len and body_len. Also fill in
message->byte_order

* dbus/dbus-string.c (_dbus_string_validate_utf8): new (not
implemented properly)
(_dbus_string_validate_nul): new function to check all-nul

* dbus/dbus-marshal.c (_dbus_marshal_get_field_end_pos): rename
get_arg_end_pos and remove all validation
(_dbus_marshal_validate_arg): actually do validation here.
ChangeLog
dbus/dbus-marshal.c
dbus/dbus-marshal.h
dbus/dbus-message.c
dbus/dbus-string.c
dbus/dbus-string.h
test/data/valid-messages/opposite-endian.message
test/data/valid-messages/simplest-manual.message
test/data/valid-messages/simplest.message