diff options
| author |   Alexander Aring <aahringo@redhat.com> | 2022-04-04 16:06:38 -0400 |
|---|---|---|
| committer |   David Teigland <teigland@redhat.com> | 2022-04-06 14:02:24 -0500 |
| commit | d9efd005fdd1d3f7ac2b9aaec025d58343d2ba45 (patch) | |
| tree | 755972aec6303d3c8801bfce1e55ed982e94fc2c | |
| parent | dlm: add __CHECKER__ for false positives (diff) | |
| download | linux-dev-d9efd005fdd1d3f7ac2b9aaec025d58343d2ba45.tar.xz linux-dev-d9efd005fdd1d3f7ac2b9aaec025d58343d2ba45.zip | |
dlm: use __le types for options header
This patch changes to use __le types directly in the dlm option headers
structures which are casted at the right dlm message buffer positions.
Currently only midcomms.c using those headers which already was calling
endian conversions on-the-fly without using in/out functionality like
other endianness handling in dlm. Using __le types now will hopefully get
useful warnings in future if we do comparison against host byte order
values.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
| -rw-r--r-- | fs/dlm/dlm_internal.h | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/fs/dlm/dlm_internal.h b/fs/dlm/dlm_internal.h index 74a9590a4dd5..2bd1b249f2ee 100644 --- a/fs/dlm/dlm_internal.h +++ b/fs/dlm/dlm_internal.h @@ -460,9 +460,9 @@ struct dlm_rcom { }; struct dlm_opt_header { - uint16_t t_type; - uint16_t t_length; - uint32_t t_pad; + __le16 t_type; + __le16 t_length; + __le32 t_pad; /* need to be 8 byte aligned */ char t_value[]; }; @@ -472,8 +472,8 @@ struct dlm_opts { struct dlm_header o_header; uint8_t o_nextcmd; uint8_t o_pad; - uint16_t o_optlen; - uint32_t o_pad2; + __le16 o_optlen; + __le32 o_pad2; char o_opts[]; }; |