【教训】12.11]梁总、白羽同时帮我看代码，寒!代码质量!!!

gofiend

                昨天下班时晚写了代码，今早测代码。发现分配内存不成功，malloc()分配1024*128失败，之后#define RD_SIZE 1024*128，改成了#define RD_SIZE 1024 ,还是不成功，把就把数组改成了静态的，       char     set_buff[RD_SIZE],comp_buff[RD_SIZE];
但是只改了这里，却忘记了改宏，之后又出错了，就是没有改按原来的动态分配所做的错误处理，就是free(set_buff);free(comp_buff);之后又出现了tcmd的coredump,两个同事看着，出这样的错的真难看。而且这个错误还是人家发现的。
还有一个错误就是return_buff,由于我是copy的原来的代码，在之前是return_buff[20];,这我攸改后的 sprintf(return_buffer,"Result of comparing  set_buff and comp_buff  is not zero, set failed!\n");已经远远超过了数量。也是梁总找出来的毛病。
还有就是运行tcmd 时先要修改权限，我却没有修改。真是的!!!
               
               
               
               
               

本文来自ChinaUnix博客，如果查看原文请点：http://blog.chinaunix.net/u2/63775/showart_1713839.html