From: "Luke Yang" <luke.adi@gmail.com>
To: "David Miller" <davem@davemloft.net>,
samuel@sortiz.org, akpm@osdl.org, linux-kernel@vger.kernel.org
Subject: [PATCH]resend: unaligned access in irda protocol stack
Date: Tue, 11 Jul 2006 11:18:24 +0800 [thread overview]
Message-ID: <489ecd0c0607102018k27890417mfdde8e112b22c48a@mail.gmail.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 3899 bytes --]
Hi all,
I decided to regenerat and resend a whold patch for this:
For "struct irda_device_info" in irda.h:
struct irda_device_info {
__u32 saddr; /* Address of local interface */
__u32 daddr; /* Address of remote device */
char info[22]; /* Description */
__u8 charset; /* Charset used for description */
__u8 hints[2]; /* Hint bits */
};
The "hints" member aligns at the third byte of a word, an odd
address. So if we visit "hints" as a short in irlmp.c and discorery.c:
u16ho(irlmp->discovery_cmd.data.hints) = irlmp->hints.word;
will cause alignment problem on some machines. Architectures with
strict alignment rules do not allow 16-bit read on an odd address.
Signed-off-by: Luke Yang <luke.adi@gmail.com>
Acked-by: David Miller <davem@davemloft.net>
discovery.c | 5 +++--
irlmp.c | 7 ++++---
2 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/net/irda/discovery.c b/net/irda/discovery.c
index 3fefc82..5ddf3e5 100644
--- a/net/irda/discovery.c
+++ b/net/irda/discovery.c
@@ -38,6 +38,7 @@ #include <net/irda/irda.h>
#include <net/irda/irlmp.h>
#include <net/irda/discovery.h>
+#include <asm/unaligned.h>
/*
* Function irlmp_add_discovery (cachelog, discovery)
@@ -86,7 +87,7 @@ void irlmp_add_discovery(hashbin_t *cach
*/
hashbin_remove_this(cachelog, (irda_queue_t *) node);
/* Check if hints bits are unchanged */
- if(u16ho(node->data.hints) == u16ho(new->data.hints))
+ if(get_unaligned(node->data.hints) ==
get_unaligned(new->data.hints))
/* Set time of first discovery for this node */
new->firststamp = node->firststamp;
kfree(node);
@@ -280,7 +281,7 @@ struct irda_device_info *irlmp_copy_disc
/* Mask out the ones we don't want :
* We want to match the discovery mask, and to get only
* the most recent one (unless we want old ones) */
- if ((u16ho(discovery->data.hints) & mask) &&
+ if ((get_unaligned(discovery->data.hints) & mask) &&
((old_entries) ||
((jiffies - discovery->firststamp) < j_timeout)) ) {
/* Create buffer as needed.
diff --git a/net/irda/irlmp.c b/net/irda/irlmp.c
index 129ad64..ee74f73 100644
--- a/net/irda/irlmp.c
+++ b/net/irda/irlmp.c
@@ -42,7 +42,6 @@ #include <net/irda/irlap.h>
#include <net/irda/iriap.h>
#include <net/irda/irlmp.h>
#include <net/irda/irlmp_frame.h>
-
#include <asm/unaligned.h>
static __u8 irlmp_find_free_slsap(void);
@@ -1063,7 +1062,7 @@ void irlmp_discovery_expiry(discinfo_t *
for(i = 0; i < number; i++) {
/* Check if we should notify client */
if ((client->expir_callback) &&
- (client->hint_mask.word & u16ho(expiries[i].hints)
+ (client->hint_mask.word &
get_unaligned(expiries[i].hints)
& 0x7f7f) )
client->expir_callback(&(expiries[i]),
EXPIRY_TIMEOUT,
@@ -1083,11 +1082,13 @@ void irlmp_discovery_expiry(discinfo_t *
*/
discovery_t *irlmp_get_discovery_response(void)
{
+ __u16 *data_hintsp;
IRDA_DEBUG(4, "%s()\n", __FUNCTION__);
IRDA_ASSERT(irlmp != NULL, return NULL;);
- u16ho(irlmp->discovery_rsp.data.hints) = irlmp->hints.word;
+ data_hintsp = (__u16 *) irlmp->discovery_rsp.data.hints;
+ put_unaligned(irlmp->hints.word, data_hintsp);
/*
* Set character set for device name (we use ASCII), and
--
Best regards,
Luke Yang
luke.adi@gmail.com
[-- Attachment #2: irda_unaligned.patch --]
[-- Type: text/x-patch, Size: 2392 bytes --]
diff --git a/net/irda/discovery.c b/net/irda/discovery.c
index 3fefc82..5ddf3e5 100644
--- a/net/irda/discovery.c
+++ b/net/irda/discovery.c
@@ -38,6 +38,7 @@ #include <net/irda/irda.h>
#include <net/irda/irlmp.h>
#include <net/irda/discovery.h>
+#include <asm/unaligned.h>
/*
* Function irlmp_add_discovery (cachelog, discovery)
@@ -86,7 +87,7 @@ void irlmp_add_discovery(hashbin_t *cach
*/
hashbin_remove_this(cachelog, (irda_queue_t *) node);
/* Check if hints bits are unchanged */
- if(u16ho(node->data.hints) == u16ho(new->data.hints))
+ if(get_unaligned(node->data.hints) == get_unaligned(new->data.hints))
/* Set time of first discovery for this node */
new->firststamp = node->firststamp;
kfree(node);
@@ -280,7 +281,7 @@ struct irda_device_info *irlmp_copy_disc
/* Mask out the ones we don't want :
* We want to match the discovery mask, and to get only
* the most recent one (unless we want old ones) */
- if ((u16ho(discovery->data.hints) & mask) &&
+ if ((get_unaligned(discovery->data.hints) & mask) &&
((old_entries) ||
((jiffies - discovery->firststamp) < j_timeout)) ) {
/* Create buffer as needed.
diff --git a/net/irda/irlmp.c b/net/irda/irlmp.c
index 129ad64..ee74f73 100644
--- a/net/irda/irlmp.c
+++ b/net/irda/irlmp.c
@@ -42,7 +42,6 @@ #include <net/irda/irlap.h>
#include <net/irda/iriap.h>
#include <net/irda/irlmp.h>
#include <net/irda/irlmp_frame.h>
-
#include <asm/unaligned.h>
static __u8 irlmp_find_free_slsap(void);
@@ -1063,7 +1062,7 @@ void irlmp_discovery_expiry(discinfo_t *
for(i = 0; i < number; i++) {
/* Check if we should notify client */
if ((client->expir_callback) &&
- (client->hint_mask.word & u16ho(expiries[i].hints)
+ (client->hint_mask.word & get_unaligned(expiries[i].hints)
& 0x7f7f) )
client->expir_callback(&(expiries[i]),
EXPIRY_TIMEOUT,
@@ -1083,11 +1082,13 @@ void irlmp_discovery_expiry(discinfo_t *
*/
discovery_t *irlmp_get_discovery_response(void)
{
+ __u16 *data_hintsp;
IRDA_DEBUG(4, "%s()\n", __FUNCTION__);
IRDA_ASSERT(irlmp != NULL, return NULL;);
- u16ho(irlmp->discovery_rsp.data.hints) = irlmp->hints.word;
+ data_hintsp = (__u16 *) irlmp->discovery_rsp.data.hints;
+ put_unaligned(irlmp->hints.word, data_hintsp);
/*
* Set character set for device name (we use ASCII), and
next reply other threads:[~2006-07-11 3:18 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-11 3:18 Luke Yang [this message]
2006-07-11 8:40 ` Samuel Ortiz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=489ecd0c0607102018k27890417mfdde8e112b22c48a@mail.gmail.com \
--to=luke.adi@gmail.com \
--cc=akpm@osdl.org \
--cc=davem@davemloft.net \
--cc=linux-kernel@vger.kernel.org \
--cc=samuel@sortiz.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®