[Xenomai-git] Paul Janzen : drivers/can: improve buffer size handling for /proc/ rtcan entries

2013-06-21 Thread git repository hosting 
Module: xenomai-forge
Branch: master
Commit: 46d70db61ecd69f5d35a90ffad1addcc00b119f0
URL:
http://git.xenomai.org/?p=xenomai-forge.git;a=commit;h=46d70db61ecd69f5d35a90ffad1addcc00b119f0

Author: Paul Janzen 
Date:   Sun Mar 17 12:16:54 2013 -0700

drivers/can: improve buffer size handling for /proc/rtcan entries

Signed-off-by: Paul Janzen 
Acked-by: Wolfgang Grandegger 

---

 kernel/drivers/can/rtcan_module.c |   26 +-
 1 files changed, 17 insertions(+), 9 deletions(-)

diff --git a/kernel/drivers/can/rtcan_module.c 
b/kernel/drivers/can/rtcan_module.c
index f270f65..6301f63 100644
--- a/kernel/drivers/can/rtcan_module.c
+++ b/kernel/drivers/can/rtcan_module.c
@@ -116,7 +116,7 @@ static void rtcan_get_timeout_name(nanosecs_rel_t timeout,
 if (timeout == RTDM_TIMEOUT_INFINITE)
strncpy(name, "infinite", max_len);
 else
-   sprintf(name, "%lld", (long long)timeout);
+   snprintf(name, max_len, "%lld", (long long)timeout);
 }
 
 static int rtcan_read_proc_devices(char *buf, char **start, off_t offset,
@@ -141,8 +141,10 @@ static int rtcan_read_proc_devices(char *buf, char 
**start, off_t offset,
 
 for (i = 1; i <= RTCAN_MAX_DEVICES; i++) {
if ((dev = rtcan_dev_get_by_index(i)) != NULL) {
-   rtcan_dev_get_state_name(dev->state, state_name, 20);
-   rtcan_dev_get_baudrate_name(dev->baudrate, baudrate_name, 20);
+   rtcan_dev_get_state_name(dev->state,
+state_name, sizeof(state_name));
+   rtcan_dev_get_baudrate_name(dev->baudrate,
+   baudrate_name, sizeof(baudrate_name));
ret = RTCAN_PROC_PRINT("%-15s %9s %-8s %10d %10d %10d\n",
   dev->name, baudrate_name, state_name,
   dev->tx_count, dev->rx_count, 
dev->err_count);
@@ -195,8 +197,10 @@ static int rtcan_read_proc_sockets(char *buf, char 
**start, off_t offset,
} else
sprintf(name, "%d", ifindex);
}
-   rtcan_get_timeout_name(sock->tx_timeout, tx_timeout, 20);
-   rtcan_get_timeout_name(sock->rx_timeout, rx_timeout, 20);
+   rtcan_get_timeout_name(sock->tx_timeout,
+  tx_timeout, sizeof(tx_timeout));
+   rtcan_get_timeout_name(sock->rx_timeout,
+  rx_timeout, sizeof(rx_timeout));
if (!RTCAN_PROC_PRINT("%2d %-15s %6d 0x%05x %13s %13s %10d %5d\n",
  context->fd, name, sock->flistlen,
  sock->err_mask, rx_timeout, tx_timeout,
@@ -224,10 +228,14 @@ static int rtcan_read_proc_info(char *buf, char **start, 
off_t offset,
 if (down_interruptible(&rtcan_devices_nrt_lock))
return -ERESTARTSYS;
 
-rtcan_dev_get_state_name(dev->state, state_name, 20);
-rtcan_dev_get_ctrlmode_name(dev->ctrl_mode, ctrlmode_name, 80);
-rtcan_dev_get_baudrate_name(dev->baudrate, baudrate_name, 20);
-rtcan_dev_get_bittime_name(&dev->bit_time, bittime_name, 80);
+rtcan_dev_get_state_name(dev->state,
+state_name, sizeof(state_name));
+rtcan_dev_get_ctrlmode_name(dev->ctrl_mode,
+   ctrlmode_name, sizeof(ctrlmode_name));
+rtcan_dev_get_baudrate_name(dev->baudrate,
+   baudrate_name, sizeof(baudrate_name));
+rtcan_dev_get_bittime_name(&dev->bit_time,
+  bittime_name, sizeof(bittime_name));
 
 if (!RTCAN_PROC_PRINT("%s %s\n", "Device", dev->name) ||
!RTCAN_PROC_PRINT("%s %s\n", "Controller", dev->ctrl_name) ||


___
Xenomai-git mailing list
Xenomai-git@xenomai.org
http://www.xenomai.org/mailman/listinfo/xenomai-git

[Xenomai-git] Paul Janzen : drivers/can: improve buffer size handling for /proc/ rtcan entries

2013-06-21 Thread git repository hosting 
Module: xenomai-forge
Branch: next
Commit: 46d70db61ecd69f5d35a90ffad1addcc00b119f0
URL:
http://git.xenomai.org/?p=xenomai-forge.git;a=commit;h=46d70db61ecd69f5d35a90ffad1addcc00b119f0

Author: Paul Janzen 
Date:   Sun Mar 17 12:16:54 2013 -0700

drivers/can: improve buffer size handling for /proc/rtcan entries

Signed-off-by: Paul Janzen 
Acked-by: Wolfgang Grandegger 

---

 kernel/drivers/can/rtcan_module.c |   26 +-
 1 files changed, 17 insertions(+), 9 deletions(-)

diff --git a/kernel/drivers/can/rtcan_module.c 
b/kernel/drivers/can/rtcan_module.c
index f270f65..6301f63 100644
--- a/kernel/drivers/can/rtcan_module.c
+++ b/kernel/drivers/can/rtcan_module.c
@@ -116,7 +116,7 @@ static void rtcan_get_timeout_name(nanosecs_rel_t timeout,
 if (timeout == RTDM_TIMEOUT_INFINITE)
strncpy(name, "infinite", max_len);
 else
-   sprintf(name, "%lld", (long long)timeout);
+   snprintf(name, max_len, "%lld", (long long)timeout);
 }
 
 static int rtcan_read_proc_devices(char *buf, char **start, off_t offset,
@@ -141,8 +141,10 @@ static int rtcan_read_proc_devices(char *buf, char 
**start, off_t offset,
 
 for (i = 1; i <= RTCAN_MAX_DEVICES; i++) {
if ((dev = rtcan_dev_get_by_index(i)) != NULL) {
-   rtcan_dev_get_state_name(dev->state, state_name, 20);
-   rtcan_dev_get_baudrate_name(dev->baudrate, baudrate_name, 20);
+   rtcan_dev_get_state_name(dev->state,
+state_name, sizeof(state_name));
+   rtcan_dev_get_baudrate_name(dev->baudrate,
+   baudrate_name, sizeof(baudrate_name));
ret = RTCAN_PROC_PRINT("%-15s %9s %-8s %10d %10d %10d\n",
   dev->name, baudrate_name, state_name,
   dev->tx_count, dev->rx_count, 
dev->err_count);
@@ -195,8 +197,10 @@ static int rtcan_read_proc_sockets(char *buf, char 
**start, off_t offset,
} else
sprintf(name, "%d", ifindex);
}
-   rtcan_get_timeout_name(sock->tx_timeout, tx_timeout, 20);
-   rtcan_get_timeout_name(sock->rx_timeout, rx_timeout, 20);
+   rtcan_get_timeout_name(sock->tx_timeout,
+  tx_timeout, sizeof(tx_timeout));
+   rtcan_get_timeout_name(sock->rx_timeout,
+  rx_timeout, sizeof(rx_timeout));
if (!RTCAN_PROC_PRINT("%2d %-15s %6d 0x%05x %13s %13s %10d %5d\n",
  context->fd, name, sock->flistlen,
  sock->err_mask, rx_timeout, tx_timeout,
@@ -224,10 +228,14 @@ static int rtcan_read_proc_info(char *buf, char **start, 
off_t offset,
 if (down_interruptible(&rtcan_devices_nrt_lock))
return -ERESTARTSYS;
 
-rtcan_dev_get_state_name(dev->state, state_name, 20);
-rtcan_dev_get_ctrlmode_name(dev->ctrl_mode, ctrlmode_name, 80);
-rtcan_dev_get_baudrate_name(dev->baudrate, baudrate_name, 20);
-rtcan_dev_get_bittime_name(&dev->bit_time, bittime_name, 80);
+rtcan_dev_get_state_name(dev->state,
+state_name, sizeof(state_name));
+rtcan_dev_get_ctrlmode_name(dev->ctrl_mode,
+   ctrlmode_name, sizeof(ctrlmode_name));
+rtcan_dev_get_baudrate_name(dev->baudrate,
+   baudrate_name, sizeof(baudrate_name));
+rtcan_dev_get_bittime_name(&dev->bit_time,
+  bittime_name, sizeof(bittime_name));
 
 if (!RTCAN_PROC_PRINT("%s %s\n", "Device", dev->name) ||
!RTCAN_PROC_PRINT("%s %s\n", "Controller", dev->ctrl_name) ||


___
Xenomai-git mailing list
Xenomai-git@xenomai.org
http://www.xenomai.org/mailman/listinfo/xenomai-git

[Xenomai-git] Paul Janzen : drivers/can: improve buffer size handling for /proc/ rtcan entries

2013-06-14 Thread git repository hosting 
Module: xenomai-forge
Branch: next
Commit: fff041d68d7ea934fdf5baabfe08c6a61af33e2d
URL:
http://git.xenomai.org/?p=xenomai-forge.git;a=commit;h=fff041d68d7ea934fdf5baabfe08c6a61af33e2d

Author: Paul Janzen 
Date:   Sun Mar 17 12:16:54 2013 -0700

drivers/can: improve buffer size handling for /proc/rtcan entries

Signed-off-by: Paul Janzen 
Acked-by: Wolfgang Grandegger 

---

 kernel/drivers/can/rtcan_module.c |   26 +-
 1 files changed, 17 insertions(+), 9 deletions(-)

diff --git a/kernel/drivers/can/rtcan_module.c 
b/kernel/drivers/can/rtcan_module.c
index f270f65..6301f63 100644
--- a/kernel/drivers/can/rtcan_module.c
+++ b/kernel/drivers/can/rtcan_module.c
@@ -116,7 +116,7 @@ static void rtcan_get_timeout_name(nanosecs_rel_t timeout,
 if (timeout == RTDM_TIMEOUT_INFINITE)
strncpy(name, "infinite", max_len);
 else
-   sprintf(name, "%lld", (long long)timeout);
+   snprintf(name, max_len, "%lld", (long long)timeout);
 }
 
 static int rtcan_read_proc_devices(char *buf, char **start, off_t offset,
@@ -141,8 +141,10 @@ static int rtcan_read_proc_devices(char *buf, char 
**start, off_t offset,
 
 for (i = 1; i <= RTCAN_MAX_DEVICES; i++) {
if ((dev = rtcan_dev_get_by_index(i)) != NULL) {
-   rtcan_dev_get_state_name(dev->state, state_name, 20);
-   rtcan_dev_get_baudrate_name(dev->baudrate, baudrate_name, 20);
+   rtcan_dev_get_state_name(dev->state,
+state_name, sizeof(state_name));
+   rtcan_dev_get_baudrate_name(dev->baudrate,
+   baudrate_name, sizeof(baudrate_name));
ret = RTCAN_PROC_PRINT("%-15s %9s %-8s %10d %10d %10d\n",
   dev->name, baudrate_name, state_name,
   dev->tx_count, dev->rx_count, 
dev->err_count);
@@ -195,8 +197,10 @@ static int rtcan_read_proc_sockets(char *buf, char 
**start, off_t offset,
} else
sprintf(name, "%d", ifindex);
}
-   rtcan_get_timeout_name(sock->tx_timeout, tx_timeout, 20);
-   rtcan_get_timeout_name(sock->rx_timeout, rx_timeout, 20);
+   rtcan_get_timeout_name(sock->tx_timeout,
+  tx_timeout, sizeof(tx_timeout));
+   rtcan_get_timeout_name(sock->rx_timeout,
+  rx_timeout, sizeof(rx_timeout));
if (!RTCAN_PROC_PRINT("%2d %-15s %6d 0x%05x %13s %13s %10d %5d\n",
  context->fd, name, sock->flistlen,
  sock->err_mask, rx_timeout, tx_timeout,
@@ -224,10 +228,14 @@ static int rtcan_read_proc_info(char *buf, char **start, 
off_t offset,
 if (down_interruptible(&rtcan_devices_nrt_lock))
return -ERESTARTSYS;
 
-rtcan_dev_get_state_name(dev->state, state_name, 20);
-rtcan_dev_get_ctrlmode_name(dev->ctrl_mode, ctrlmode_name, 80);
-rtcan_dev_get_baudrate_name(dev->baudrate, baudrate_name, 20);
-rtcan_dev_get_bittime_name(&dev->bit_time, bittime_name, 80);
+rtcan_dev_get_state_name(dev->state,
+state_name, sizeof(state_name));
+rtcan_dev_get_ctrlmode_name(dev->ctrl_mode,
+   ctrlmode_name, sizeof(ctrlmode_name));
+rtcan_dev_get_baudrate_name(dev->baudrate,
+   baudrate_name, sizeof(baudrate_name));
+rtcan_dev_get_bittime_name(&dev->bit_time,
+  bittime_name, sizeof(bittime_name));
 
 if (!RTCAN_PROC_PRINT("%s %s\n", "Device", dev->name) ||
!RTCAN_PROC_PRINT("%s %s\n", "Controller", dev->ctrl_name) ||


___
Xenomai-git mailing list
Xenomai-git@xenomai.org
http://www.xenomai.org/mailman/listinfo/xenomai-git