Have radmsg_copy_attrs() return error in all error cases.

Also when copying of the first attribute fails.

1 files changed, 2 insertions, 4 deletions

diff --git a/radmsg.c b/radmsg.c
index 6e33990..7f6dd9d 100644
--- a/radmsg.c
+++ b/radmsg.c
@@ -106,9 +106,7 @@ struct tlv *radmsg_gettype(struct radmsg *msg, uint8_t type) {
  * If all attributes were copied successfully, the number of
  * attributes copied is returned.
  *
- * If copying failed, a negative number is returned. The number
- * returned is 0 minus the number of attributes successfully copied
- * before the failure. */
+ * If copying failed, a negative number is returned. */
 int radmsg_copy_attrs(struct radmsg *dst,
                       const struct radmsg *src,
                       uint8_t type)
@@ -119,7 +117,7 @@ int radmsg_copy_attrs(struct radmsg *dst,
 
     for (node = list_first(list); node; node = list_next(node)) {
         if (radmsg_add(dst, copytlv((struct tlv *) node->data)) != 1) {
-            n = -n;
+            n = -1;
             break;
         }
         n++;