[dia] [scan-build] Null dereference (potentially)
- From: Hans Breuer <hans src gnome org>
- To: svn-commits-list gnome org
- Cc:
- Subject: [dia] [scan-build] Null dereference (potentially)
- Date: Sun, 13 Dec 2009 19:12:07 +0000 (UTC)
commit 628b661cf44ab9ba96607cbc79a338379fff19d6
Author: Hans Breuer <hans breuer org>
Date: Sun Dec 13 13:41:19 2009 +0100
[scan-build] Null dereference (potentially)
Checking param->name before param->name[0] although the
GUI would not allow the case to happen. An empty param->name
still is "".
objects/UML/umloperation.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
---
diff --git a/objects/UML/umloperation.c b/objects/UML/umloperation.c
index 6e43444..c5844cd 100644
--- a/objects/UML/umloperation.c
+++ b/objects/UML/umloperation.c
@@ -337,7 +337,7 @@ uml_get_operation_string (UMLOperation *operation)
len += (param->name ? strlen (param->name) : 0);
if (param->type != NULL) {
len += strlen (param->type);
- if (param->type[0] && param->name[0]) {
+ if (param->type[0] && (param->name != NULL && param->name[0])) {
len += 1;
}
}
@@ -396,7 +396,7 @@ uml_get_operation_string (UMLOperation *operation)
strcat (str, param->name ? param->name : "");
if (param->type != NULL) {
- if (param->type[0] && param->name[0]) {
+ if (param->type[0] && (param->name != NULL && param->name[0])) {
strcat (str, ":");
}
strcat (str, param->type);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]