diff options
| author | Tamito Kajiyama <rd6t-kjym@asahi-net.or.jp> | 2013-03-02 22:43:06 +0400 |
|---|---|---|
| committer | Tamito Kajiyama <rd6t-kjym@asahi-net.or.jp> | 2013-03-02 22:43:06 +0400 |
| commit | 51460e7789822bc6fd42ab7fed3fe9c84c9c27f4 (patch) | |
| tree | adc01144200ca9ea39da66402c9fc2f361487ae1 /source/blender/freestyle/intern | |
| parent | 6645551aedf6c6e61094f8a5eb70efec89d4e19b (diff) | |
Fix for a buffer overflow, by simply removing the unnecessary string copy.
Review comment from Campbell.
Diffstat (limited to 'source/blender/freestyle/intern')
| -rw-r--r-- | source/blender/freestyle/intern/blender_interface/BlenderStrokeRenderer.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/source/blender/freestyle/intern/blender_interface/BlenderStrokeRenderer.cpp b/source/blender/freestyle/intern/blender_interface/BlenderStrokeRenderer.cpp index 51739cd4720..254aca35d82 100644 --- a/source/blender/freestyle/intern/blender_interface/BlenderStrokeRenderer.cpp +++ b/source/blender/freestyle/intern/blender_interface/BlenderStrokeRenderer.cpp @@ -175,8 +175,7 @@ BlenderStrokeRenderer::~BlenderStrokeRenderer() for (Base *b = (Base*)freestyle_scene->base.first; b; b = b->next) { Object *ob = b->object; void *data = ob->data; - char name[24]; - strcpy(name, ob->id.name); + char *name = ob->id.name; #if 0 if (G.debug & G_DEBUG_FREESTYLE) { cout << "removing " << name[0] << name[1] << ":" << (name+2) << endl; |