summaryrefslogtreecommitdiffstats
path: root/cpukit/libnetworking/net/ppp_tty.c
diff options
context:
space:
mode:
authorJoel Sherrill <joel.sherrill@OARcorp.com>2002-03-27 14:36:07 +0000
committerJoel Sherrill <joel.sherrill@OARcorp.com>2002-03-27 14:36:07 +0000
commitf4c118d7a32370516b0871920b36002895f14f1d (patch)
treed071b48cfc44a1f0e9c7aab43489f0507caa3eb1 /cpukit/libnetworking/net/ppp_tty.c
parent90f30c8c145117851c10bda31235fac1b4fdd8a7 (diff)
downloadrtems-f4c118d7a32370516b0871920b36002895f14f1d.tar.bz2
2002-03-27 Ilya Alexeev <ilya@continuum.ru>
* PR162 * net/if_ppp.c ppp_txdaemon(), net/if_pppvar.h pppstart(): Local variables must not be used in a device write routines. Now ppp_softc structure have own character for writing to device (sc_outchar). I think that converting local variables to static is not a right solution, because problems will occur in the case of two or more ppp instances. * net/ppp_tty.c pppstart(): Type of the ioffset variable must be u_long, otherwise in the case of the big output packet endless loop may occur.
Diffstat (limited to 'cpukit/libnetworking/net/ppp_tty.c')
-rw-r--r--cpukit/libnetworking/net/ppp_tty.c21
1 files changed, 10 insertions, 11 deletions
diff --git a/cpukit/libnetworking/net/ppp_tty.c b/cpukit/libnetworking/net/ppp_tty.c
index b655fd4f04..060bd3976c 100644
--- a/cpukit/libnetworking/net/ppp_tty.c
+++ b/cpukit/libnetworking/net/ppp_tty.c
@@ -562,10 +562,8 @@ pppasyncctlp(sc)
int
pppstart(struct rtems_termios_tty *tp)
{
- char c;
char *sendBegin;
- char cFrame = (char )PPP_FLAG;
- u_char ioffset = (u_char )0;
+ u_long ioffset = (u_long )0;
struct mbuf *m = (struct mbuf *)0;
struct ppp_softc *sc = tp->t_sc;
@@ -596,7 +594,8 @@ pppstart(struct rtems_termios_tty *tp)
else {
/* done with this packet */
sc->sc_outflag &= ~SC_TX_BUSY;
- (*tp->device.write)(tp->minor, &cFrame, 1);
+ sc->sc_outchar = (u_char)PPP_FLAG;
+ (*tp->device.write)(tp->minor, &sc->sc_outchar, 1);
rtems_event_send(sc->sc_txtask, TX_TRANSMIT);
}
}
@@ -604,11 +603,11 @@ pppstart(struct rtems_termios_tty *tp)
/* check to see if there is some data to write out */
if ( sc->sc_outoff < sc->sc_outlen ) {
/* check to see if character needs to be escaped */
- c = sc->sc_outbuf[sc->sc_outoff];
- if ( ESCAPE_P(c) ) {
+ sc->sc_outchar = sc->sc_outbuf[sc->sc_outoff];
+ if ( ESCAPE_P(sc->sc_outchar) ) {
if ( sc->sc_outflag & SC_TX_ESCAPE ) {
/* last sent character was the escape character */
- c = c ^ PPP_TRANS;
+ sc->sc_outchar = sc->sc_outchar ^ PPP_TRANS;
/* clear the escape flag and increment the offset */
sc->sc_outflag &= ~SC_TX_ESCAPE;
@@ -616,18 +615,18 @@ pppstart(struct rtems_termios_tty *tp)
}
else {
/* need to send the escape character */
- c = PPP_ESCAPE;
+ sc->sc_outchar = PPP_ESCAPE;
/* set the escape flag */
sc->sc_outflag |= SC_TX_ESCAPE;
}
- sendBegin = &c;
+ sendBegin = &sc->sc_outchar;
}
else {
/* escape not needed - increment the offset as much as possible */
- while ((!ESCAPE_P(c)) && ((sc->sc_outoff + ioffset) < sc->sc_outlen)) {
+ while ((!ESCAPE_P(sc->sc_outchar)) && ((sc->sc_outoff + ioffset) < sc->sc_outlen)) {
ioffset++;
- c = sc->sc_outbuf[sc->sc_outoff + ioffset];
+ sc->sc_outchar = sc->sc_outbuf[sc->sc_outoff + ioffset];
}
sendBegin = &sc->sc_outbuf[sc->sc_outoff];
}