diff options
author | glebius <glebius@FreeBSD.org> | 2005-05-16 19:12:57 +0000 |
---|---|---|
committer | glebius <glebius@FreeBSD.org> | 2005-05-16 19:12:57 +0000 |
commit | 27540c786b78ab1c631cf4e380b874e6cb9dd70b (patch) | |
tree | 08c783c85a90e1e88f13434db1df8ae13a5be0ad /sys/netgraph/ng_ppp.c | |
parent | 8003014f216927129ca9d82bdc33396bfe5abc33 (diff) | |
download | FreeBSD-src-27540c786b78ab1c631cf4e380b874e6cb9dd70b.zip FreeBSD-src-27540c786b78ab1c631cf4e380b874e6cb9dd70b.tar.gz |
Refactor places where ng_package_data() failed.
Suggested & reviewed by: archie
Diffstat (limited to 'sys/netgraph/ng_ppp.c')
-rw-r--r-- | sys/netgraph/ng_ppp.c | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/sys/netgraph/ng_ppp.c b/sys/netgraph/ng_ppp.c index fb48eeb..7016a73 100644 --- a/sys/netgraph/ng_ppp.c +++ b/sys/netgraph/ng_ppp.c @@ -1319,18 +1319,17 @@ ng_ppp_frag_process(node_p node) /* Deliver any deliverable packets */ while (ng_ppp_check_packet(node)) { ng_ppp_get_packet(node, &m); - if ((item = ng_package_data(m, NG_NOFLAGS)) == NULL) - return (ENOMEM); - ng_ppp_input(node, 0, NG_PPP_BUNDLE_LINKNUM, item); + if ((item = ng_package_data(m, NG_NOFLAGS)) != NULL) + ng_ppp_input(node, 0, NG_PPP_BUNDLE_LINKNUM, item); } /* Delete dead fragments and try again */ if (ng_ppp_frag_trim(node)) { while (ng_ppp_check_packet(node)) { ng_ppp_get_packet(node, &m); - if ((item = ng_package_data(m, NG_NOFLAGS)) == NULL) - return (ENOMEM); - ng_ppp_input(node, 0, NG_PPP_BUNDLE_LINKNUM, item); + if ((item = ng_package_data(m, NG_NOFLAGS)) != NULL) + ng_ppp_input(node, 0, NG_PPP_BUNDLE_LINKNUM, + item); } } @@ -1600,13 +1599,14 @@ deliver: } /* Send fragment */ - if ((item = ng_package_data(m2, NG_NOFLAGS)) == NULL) - return (ENOMEM); - error = ng_ppp_output(node, 0, PROT_MP, linkNum, item); - if (error != 0) { - if (!lastFragment) - NG_FREE_M(m); - return (error); + if ((item = ng_package_data(m2, NG_NOFLAGS)) != NULL) { + error = ng_ppp_output(node, 0, PROT_MP, + linkNum, item); + if (error != 0) { + if (!lastFragment) + NG_FREE_M(m); + return (error); + } } } } |