diff options
author | Campbell Barton <ideasman42@gmail.com> | 2011-02-13 06:21:27 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2011-02-13 06:21:27 +0300 |
commit | 867fc4b463ef39ea16103f18f332c3d259624d29 (patch) | |
tree | 7d20c416241afb7b878b767a9955e284d3cddbe2 /source/blender/render | |
parent | 9e03a0d4762b4734fe7ccb20e03b4a3c8f939620 (diff) |
enforce string limits (reported by pedantic checking tools & some developers).
mostly replace strcpy with BLI_strncpy and multiple strcat's with a BLI_snprintf().
also fix possible crash if CWD isnt available.
Diffstat (limited to 'source/blender/render')
-rw-r--r-- | source/blender/render/intern/source/pipeline.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/source/blender/render/intern/source/pipeline.c b/source/blender/render/intern/source/pipeline.c index a3f98318444..dc5d5b0693a 100644 --- a/source/blender/render/intern/source/pipeline.c +++ b/source/blender/render/intern/source/pipeline.c @@ -574,7 +574,7 @@ static RenderResult *new_render_result(Render *re, rcti *partrct, int crop, int rl= MEM_callocN(sizeof(RenderLayer), "new render layer"); BLI_addtail(&rr->layers, rl); - strcpy(rl->name, srl->name); + BLI_strncpy(rl->name, srl->name, sizeof(rl->name)); rl->lay= srl->lay; rl->lay_zmask= srl->lay_zmask; rl->layflag= srl->layflag; @@ -921,7 +921,7 @@ static void renderresult_add_names(RenderResult *rr) for(rl= rr->layers.first; rl; rl= rl->next) for(rpass= rl->passes.first; rpass; rpass= rpass->next) - strcpy(rpass->name, get_pass_name(rpass->passtype, -1)); + BLI_strncpy(rpass->name, get_pass_name(rpass->passtype, -1), sizeof(rpass->name)); } /* called for reading temp files, and for external engines */ @@ -2049,7 +2049,7 @@ static void load_backbuffer(Render *re) ImBuf *ibuf; char name[256]; - strcpy(name, re->r.backbuf); + BLI_strncpy(name, re->r.backbuf, sizeof(name)); BLI_path_abs(name, re->main->name); BLI_path_frame(name, re->r.cfra, 0); |