diff options
author | Allan Stephens <allan.stephens@windriver.com> | 2008-07-15 01:44:01 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-07-15 01:44:01 -0400 |
commit | 0e35fd5e5264bb46d1febbe9cd9aa08421c21a96 (patch) | |
tree | 984ea14bf1e691d02b3202abeff087ba4369bc44 /net/tipc/bcast.c | |
parent | 2da59918e26837f305131cfac9c0f1b3b42bb8ae (diff) |
tipc: Eliminate improper use of TIPC_OK error code
This patch corrects many places where TIPC routines indicated
successful completion by returning TIPC_OK instead of 0.
(The TIPC_OK symbol has the value 0, but it should only be used
in contexts that deal with the error code field of a TIPC
message header.)
Signed-off-by: Allan Stephens <allan.stephens@windriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/bcast.c')
-rw-r--r-- | net/tipc/bcast.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/net/tipc/bcast.c b/net/tipc/bcast.c index a5883b1452ff..b1ff16aa4bdb 100644 --- a/net/tipc/bcast.c +++ b/net/tipc/bcast.c | |||
@@ -611,7 +611,7 @@ swap: | |||
611 | bcbearer->bpairs[bp_index].secondary = p; | 611 | bcbearer->bpairs[bp_index].secondary = p; |
612 | update: | 612 | update: |
613 | if (bcbearer->remains_new.count == 0) | 613 | if (bcbearer->remains_new.count == 0) |
614 | return TIPC_OK; | 614 | return 0; |
615 | 615 | ||
616 | bcbearer->remains = bcbearer->remains_new; | 616 | bcbearer->remains = bcbearer->remains_new; |
617 | } | 617 | } |
@@ -620,7 +620,7 @@ update: | |||
620 | 620 | ||
621 | bcbearer->bearer.publ.blocked = 1; | 621 | bcbearer->bearer.publ.blocked = 1; |
622 | bcl->stats.bearer_congs++; | 622 | bcl->stats.bearer_congs++; |
623 | return ~TIPC_OK; | 623 | return 1; |
624 | } | 624 | } |
625 | 625 | ||
626 | /** | 626 | /** |
@@ -756,7 +756,7 @@ int tipc_bclink_reset_stats(void) | |||
756 | spin_lock_bh(&bc_lock); | 756 | spin_lock_bh(&bc_lock); |
757 | memset(&bcl->stats, 0, sizeof(bcl->stats)); | 757 | memset(&bcl->stats, 0, sizeof(bcl->stats)); |
758 | spin_unlock_bh(&bc_lock); | 758 | spin_unlock_bh(&bc_lock); |
759 | return TIPC_OK; | 759 | return 0; |
760 | } | 760 | } |
761 | 761 | ||
762 | int tipc_bclink_set_queue_limits(u32 limit) | 762 | int tipc_bclink_set_queue_limits(u32 limit) |
@@ -769,7 +769,7 @@ int tipc_bclink_set_queue_limits(u32 limit) | |||
769 | spin_lock_bh(&bc_lock); | 769 | spin_lock_bh(&bc_lock); |
770 | tipc_link_set_queue_limits(bcl, limit); | 770 | tipc_link_set_queue_limits(bcl, limit); |
771 | spin_unlock_bh(&bc_lock); | 771 | spin_unlock_bh(&bc_lock); |
772 | return TIPC_OK; | 772 | return 0; |
773 | } | 773 | } |
774 | 774 | ||
775 | int tipc_bclink_init(void) | 775 | int tipc_bclink_init(void) |
@@ -810,7 +810,7 @@ int tipc_bclink_init(void) | |||
810 | tipc_printbuf_init(&bcl->print_buf, pb, BCLINK_LOG_BUF_SIZE); | 810 | tipc_printbuf_init(&bcl->print_buf, pb, BCLINK_LOG_BUF_SIZE); |
811 | } | 811 | } |
812 | 812 | ||
813 | return TIPC_OK; | 813 | return 0; |
814 | } | 814 | } |
815 | 815 | ||
816 | void tipc_bclink_stop(void) | 816 | void tipc_bclink_stop(void) |