| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
|
|
|
|
| |
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
|
|
|
|
| |
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
|
|
|
|
|
|
|
|
|
|
| |
gcc -DHAVE_CONFIG_H -I. -I.. -O2 -Wall -Wno-pointer-sign -MT call.o -MD -MP
-MF .deps/call.Tpo -c -o call.o call.c
call.c: In function ‘wrdstatw’:
call.c:482:9: warning: variable ‘x’ set but not used [-Wunused-but-set-variable]
mv -f .deps/call.Tpo .deps/call.Po
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
|
|
|
|
|
|
|
|
|
|
| |
gcc -DHAVE_CONFIG_H -I. -I.. -O2 -Wall -Wno-pointer-sign -MT menu.o -MD -MP -MF .deps/menu.Tpo -c -o menu.o menu.c
menu.c: In function ‘p_dwn_menu’:
menu.c:132:15: warning: variable ‘pos’ set but not used [-Wunused-but-set-variable]
menu.c: In function ‘top_menu’:
menu.c:244:11: warning: variable ‘pos’ set but not used [-Wunused-but-set-variable]
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
|
|
|
|
|
|
|
| |
call.c:1590:6: warning: variable ‘extrach’ set but not used [-Wunused-but-set-va
riable]
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
|
|
|
|
| |
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
|
|
|
|
| |
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
If the call sign you connect to has 9 characters
(i.E. DL9SAU-10), then call segfaults. idString
in the ncruses mode routine start_screen() was
dimensioned too short, and the auther had the fatal
assumtion that a call is 8 characters long (and not 9),
and he did limit the length in sprintf with ... %n.ns
This is an old bug. Obviously, ubuntu 12.04 and 12.10
introduced better runtime checks into potential buffer
overflows.
|
|
|
|
|
|
| |
For now these options are just honored for the call.1 and listen.1 man
pages because these days these two programs generally get renamed to
avoid naming conflicts with other programs.
|
|
|
|
|
|
|
|
|
| |
- Better error handling on read() == -1 and write() == -1.
Previously, the return value was not honored correctly - and could
cause unexpected results.
- New option "-R":
Disable all remote commands like autobin (#BIN#), 7plus downloads, //echo, etc.
This is useful for scripting where you just don't like the remote site to trigger uncaught conditions here.
|
|
|
|
|
|
|
|
| |
read() = 0, wht consumed 100% CPU power.
New options:
-W: waits after EOF on stdin. Useful for scripting, for waiting
for data coming via fd 3 (ax25 connection)
-T secs: idle timout after secs seconds.
|
|
|
|
|
|
| |
Glibc as old as 2.2 provides working headers under the right name and
if not libax25 would install headers under the right name so this has
become redundant and ugly.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Keeping generated files in CVS doesn't only tend to produce large and
cluttered files it also may result in build problems due to timestamps
in the wrong order. So dump everything, update .cvsignore to ignore
these files and resolve all warnings generated by autoreconf.
From now on users of a CVS checkout should run the command
autoreconf --install --force
after having done a CVS checkout. For this to succeed automake, autoconf
and libtool will have to be installed.
|
| |
|
|
|
|
|
|
|
|
|
| |
wnoutrefresh() [ncurses functions], called from winclose() in menu.c.
Debunging with gdb showed that wtab->fline was a large negative number.
Now doing better checking that wtab->fline and wtab->lline is > 0.
Seems that this fix is sufficient. If it will come out that it is not,
we'll write a wtouchln() wrapper which checks if the arguments are
positive and reasonable small.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
from the ax25 connection using write(0). everything was fine
as long as call wrote to stdout.
If instead we use output redirection to a file or stdout
(> foo, | bar), then the messages like "Connected to.." appeared
at the end of the connection.
I guess it's a glibc issue that printf (whih is buffered) behaves
different when stdout is redirected.
Fixed using fflush() after each printf().
|
| |
|
| |
|
| |
|
|
|
|
| |
issue whenever a packet ended with "\r ".
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. if a line was split over two ax25 packets, we read i.e.
1. "foobar told me" 2. " go7+. ". The second packet was
interpreted like starting with " go7+. ".
thus in a pure bbs listing it was misinterpreted as start
of a 7plus file.
Since the data was not like a 7plus header starts, the
7plus download parser caused a segfault (after copying
i.E. 1555 bytes to char s[20]. [seen in gdb ;-]
-> enforced a "linemode".
2. protocol and array size assurances in the #BIN and 7plus
part, as well as in dupdstatw().
|
| |
|
| |
|
| |
|
|
|
|
|
| |
currently we've seen other bugs in call which lead to malfunctions
and segfaults.
|
| |
|
|
|
|
| |
fixed dos to unix time conversion.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
- fileupload (in raw and gui "slave" mode) goes only
step by step the 128 bytes further if user enters a character.
Thanks to Daniel DO8CD for the report.
- write() returns -1 at EAGAIN (packet links are usually slower
then linux file IO, and EAGAIN occurs because the txbuffer
in the kernel is full). This lead to a bad substraction in
sent bytes and thus to file corruption at the receiving site.
- added more help: now explaining all the ~-Escapes in raw mode
in the ~? command.
|
|
|
|
|
|
| |
- call was not usable as a pipe in scripts because it read 511
bytes, tried to write these 511 bytes (but 256 is max for ax25
I-frames), got -1 EMSGSIZE (Message too long) and terminated.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. if stdin closes (i.e. if call is right end of a pipe),
then call should close. this also fixes the case when
call reads from a unix fifo, and the other end of the
fifo terminates. This produced highest cpu-load,
with select(), read() = 0, select(), read() = 0, ..
2. if stdin is not a tty, i.e. call is called from ax25d,
then it's not a good idea to honor the ~ commands.
In that special case, you could imagine what ~! means,
while ax25d is running as uid 0...
3. while not operating on pty, the FILE buffers are not
flushed in all cases. Thus, the messages like "Connected .."
were be printed in the termination phase of the program..
|
| |
|
| |
|
|
|
|
| |
- usage: new function usage()
|
|
|
|
| |
More autobbreak updating nightmares
|
| |
|
|
|
|
| |
* Added \n to poll line
|
| |
|
|
|
|
| |
Makefile bug
|
| |
|
| |
|
| |
|
| |
|
|
|