summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSebastian Huber <sebastian.huber@embedded-brains.de>2010-08-25 08:31:43 +0000
committerSebastian Huber <sebastian.huber@embedded-brains.de>2010-08-25 08:31:43 +0000
commit6ef96da17e8ef6d92888c9a6f21b558a7d480e64 (patch)
treedccd07f4197d2b35b2c046bfd9063ae700f299c0
parent2010-08-25 Sebastian Huber <sebastian.huber@embedded-brains.de> (diff)
downloadrtems-6ef96da17e8ef6d92888c9a6f21b558a7d480e64.tar.bz2
2010-08-25 Sebastian Huber <sebastian.huber@embedded-brains.de>
CID 114/SECURE_CODING * libblock/src/media.c: Avoid strcpy().
-rw-r--r--cpukit/ChangeLog5
-rw-r--r--cpukit/libblock/src/media.c2
2 files changed, 6 insertions, 1 deletions
diff --git a/cpukit/ChangeLog b/cpukit/ChangeLog
index 2ebde0db19..eb8db8cb42 100644
--- a/cpukit/ChangeLog
+++ b/cpukit/ChangeLog
@@ -1,5 +1,10 @@
2010-08-25 Sebastian Huber <sebastian.huber@embedded-brains.de>
+ CID 114/SECURE_CODING
+ * libblock/src/media.c: Avoid strcpy().
+
+2010-08-25 Sebastian Huber <sebastian.huber@embedded-brains.de>
+
CID 113/SECURE_CODING
* libblock/src/media-path.c: Avoid strcpy(). Style. Use assert() to
avoid dead code.
diff --git a/cpukit/libblock/src/media.c b/cpukit/libblock/src/media.c
index 1e62a7bd09..b4ddd6ad7c 100644
--- a/cpukit/libblock/src/media.c
+++ b/cpukit/libblock/src/media.c
@@ -244,7 +244,7 @@ static void create_item(
item->parent = parent;
item->disk_path = (char *) item + sizeof(*item);
- strcpy(item->disk_path, disk_path);
+ memcpy(item->disk_path, disk_path, disk_path_size);
rtems_chain_append(&media_item_chain, &item->node);
}
}