* [PATCH] xen-netback: make ops structs const
@ 2012-01-04 21:56 Stephen Hemminger
2012-01-05 8:44 ` Ian Campbell
2012-01-05 18:23 ` David Miller
0 siblings, 2 replies; 3+ messages in thread
From: Stephen Hemminger @ 2012-01-04 21:56 UTC (permalink / raw)
To: Ian Campbell; +Cc: xen-devel, netdev
All tables of function pointers should be const to make hacks
more difficult. Compile tested only.
Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
---
Patch against net-next
--- a/drivers/net/xen-netback/interface.c 2011-12-07 10:54:19.232282492 -0800
+++ b/drivers/net/xen-netback/interface.c 2012-01-04 13:16:04.414052721 -0800
@@ -223,7 +223,7 @@ static void xenvif_get_strings(struct ne
}
}
-static struct ethtool_ops xenvif_ethtool_ops = {
+static const struct ethtool_ops xenvif_ethtool_ops = {
.get_link = ethtool_op_get_link,
.get_sset_count = xenvif_get_sset_count,
@@ -231,7 +231,7 @@ static struct ethtool_ops xenvif_ethtool
.get_strings = xenvif_get_strings,
};
-static struct net_device_ops xenvif_netdev_ops = {
+static const struct net_device_ops xenvif_netdev_ops = {
.ndo_start_xmit = xenvif_start_xmit,
.ndo_get_stats = xenvif_get_stats,
.ndo_open = xenvif_open,
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] xen-netback: make ops structs const
2012-01-04 21:56 [PATCH] xen-netback: make ops structs const Stephen Hemminger
@ 2012-01-05 8:44 ` Ian Campbell
2012-01-05 18:23 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: Ian Campbell @ 2012-01-05 8:44 UTC (permalink / raw)
To: Stephen Hemminger; +Cc: xen-devel@lists.xensource.com, netdev@vger.kernel.org
On Wed, 2012-01-04 at 21:56 +0000, Stephen Hemminger wrote:
> All tables of function pointers should be const to make hacks
> more difficult. Compile tested only.
I also eyeballed for any writes to these structs and couldn't find one.
> Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
>
> ---
> Patch against net-next
>
> --- a/drivers/net/xen-netback/interface.c 2011-12-07 10:54:19.232282492 -0800
> +++ b/drivers/net/xen-netback/interface.c 2012-01-04 13:16:04.414052721 -0800
> @@ -223,7 +223,7 @@ static void xenvif_get_strings(struct ne
> }
> }
>
> -static struct ethtool_ops xenvif_ethtool_ops = {
> +static const struct ethtool_ops xenvif_ethtool_ops = {
> .get_link = ethtool_op_get_link,
>
> .get_sset_count = xenvif_get_sset_count,
> @@ -231,7 +231,7 @@ static struct ethtool_ops xenvif_ethtool
> .get_strings = xenvif_get_strings,
> };
>
> -static struct net_device_ops xenvif_netdev_ops = {
> +static const struct net_device_ops xenvif_netdev_ops = {
> .ndo_start_xmit = xenvif_start_xmit,
> .ndo_get_stats = xenvif_get_stats,
> .ndo_open = xenvif_open,
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] xen-netback: make ops structs const
2012-01-04 21:56 [PATCH] xen-netback: make ops structs const Stephen Hemminger
2012-01-05 8:44 ` Ian Campbell
@ 2012-01-05 18:23 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2012-01-05 18:23 UTC (permalink / raw)
To: shemminger; +Cc: ian.campbell, xen-devel, netdev
From: Stephen Hemminger <shemminger@vyatta.com>
Date: Wed, 4 Jan 2012 13:56:58 -0800
> All tables of function pointers should be const to make hacks
> more difficult. Compile tested only.
>
> Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Applied.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-01-05 18:23 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-04 21:56 [PATCH] xen-netback: make ops structs const Stephen Hemminger
2012-01-05 8:44 ` Ian Campbell
2012-01-05 18:23 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).