View Issue Details

IDProjectCategoryView StatusLast Update
0000568Other Unix PortBugpublic2020-08-24 03:11
ReporterLewisR Assigned Topsmedley  
PrioritylowSeverityminorReproducibilityalways
Status closedResolutionfixed 
Summary0000568: ncFTP crashes when "?" passed to the running instance as a command
DescriptionLOL...

Still learning my way around ncFTP. At a prompt in the app (connected as anonymous to my FTP server, running Peter's daemon 1.345, now with UTF-8 support), I typed "?" to get help (I hoped). Instead, I got:

Assertion failed: gNumCommands < (sizeof(cmdnames) / sizeof(char *)), file cmds.
c, line 1384

Killed by SIGABRT
pid=0x00cc ppid=0x00af tid=0x0001 slot=0x00cf pri=0x0200 mc=0x0001
J:\PROGRAMS\NCFTP\NCFTP.EXE
Process dumping was disabled, use DUMPPROC / PROCDUMP to enable it.
Steps To Reproduce1. Start ncFTP (it is not necessary to connect to a server).
2. Type "?" (without quotes), and press <Enter>.
Additional InformationI hadn't noticed this before. Perhaps it's some updated library on my ThinkPad.
TagsNo tags attached.

Activities

LewisR

2013-12-01 19:46

developer   ~0002584

I should clarify that this was with ncFTP 3.2.2 (2008-11-02).

psmedley

2013-12-16 08:22

administrator   ~0002587

I built 3.2.5 and it seems to work ok.

http://smedley.id.au/tmp/ncftp-3.2.5-os2-20131216.zip

Note I only tested help, didn't make sure anything else worked :P

LewisR

2013-12-16 15:49

developer   ~0002588

Elbert just built this, using your diff from 3.2.2, and help seemed to work without an issue. I'll test your build and compare my experience. Essentially, I noticed some path issues, particularly (and this I saw with your previous build, Paul) when running local commands, which are hardcoded to look in /usr/bin for things like mkdir, ls, rm, etc. The only way to satisfy them is to actually have a /bin directory with executables in %UNIXROOT%. The other issue I saw with Elbert's build (and under Linux, as well) was case-sensitivity with tab completion, thus, with a directory containing:

/warpstock
/warpevents
/Warpstock.txt

running the command:

ls /war <tab>

the results are:

warpstock
warpevents

which is an annoyance (and again, consistent with the Linux build).

I'll see if I have the same experiences with your latest one. Thanks!

LewisR

2013-12-16 16:10

developer   ~0002589

Still some drive letter issues:

[j:\usr\local\bin].\ncftp.exe
Who are you?
You have a user id number of 0, but no username associated with it.
NcFTP 3.2.5 (Feb 02, 2011) by Mike Gleason (http://www.NcFTP.com/contact/).
ncftp> ?
Commands may be abbreviated. 'help showall' shows hidden and unsupported
commands. 'help <command>' gives a brief description of <command>.

ascii cat get lookup mkdir pwd set
bgget cd help lpage open quit show
bgput chmod jobs lpwd page quote site
bgstart close lcd lrename passive rename type
binary debug lchmod lrm pdir rhelp umask
bookmark dir lls lrmdir pls rm version
bookmarks edit lmkdir ls put rmdir

For details, please see the manual ("man ncftp" at your regular shell prompt
or online at http://www.NcFTP.com/ncftp/doc/ncftp.html).
ncftp> lls
Unknown command "J:"

(There actually *is* a /bin directory on this drive - %UNIXROOT% is J: on this machine - and ls.exe is present in that directory.)

psmedley

2013-12-21 09:03

administrator   ~0002591

http://smedley.id.au/tmp/ncftp-3.2.5-os2-20131221.zip

lls should execute ls.exe from path

psmedley

2013-12-21 22:42

administrator   ~0002592

not tested, but hopefully http://smedley.id.au/ncftp-3.2.5-os2-20131222.zip will ignore case on filename completion

LewisR

2013-12-21 23:20

developer   ~0002593

Writing up a new bug, Paul, as this issue should be closed. ;-)

BTW, the url above is broken (you missed the /tmp in between).

psmedley

2013-12-22 08:14

administrator   ~0002595

Fixed in 3.2.5

Issue History

Date Modified Username Field Change
2013-04-07 20:42 LewisR New Issue
2013-12-01 19:46 LewisR Note Added: 0002584
2013-12-16 08:22 psmedley Note Added: 0002587
2013-12-16 15:49 LewisR Note Added: 0002588
2013-12-16 16:10 LewisR Note Added: 0002589
2013-12-21 09:03 psmedley Note Added: 0002591
2013-12-21 22:42 psmedley Note Added: 0002592
2013-12-21 23:20 LewisR Note Added: 0002593
2013-12-22 08:14 psmedley Note Added: 0002595
2013-12-22 08:14 psmedley Status new => resolved
2013-12-22 08:14 psmedley Resolution open => fixed
2013-12-22 08:14 psmedley Assigned To => psmedley
2020-08-24 03:11 psmedley Status resolved => closed