top的Bug

发现了Ubuntu 10.10下的procps 3.2.8-9ubuntu3的top的一个Bug：

$ top -U tux

    top: unknown argument 'O'

usage:    top -hv | -bcisSH -d delay -n iterations [-u user | -U user] -p pid [,pid ...]

发现只要是-U后的用户名存在，那么就出这样莫名其妙的错误。

后来简单printf研究了一阵子后，打算用GDB。需要在procps的Makefile里面，把CFLAGS变量里面的-s去掉，加上-g选项，或者-ggdb选项。GCC中这两个参数的含义是：

-s  Remove all symbol table and relocation information from the

           executable.

-g  Produce debugging information in the operating system's native

           format (stabs, COFF, XCOFF, or DWARF 2).  GDB can work with this

           debugging information.

通过GDB研究程序运行流程发现，top的parse_args函数在处理完-U参数的时候，本来应该指向字符串末尾'\0'的指针指到了后面无关的区域，又进行了一边参数处理，开头的unknown argument 'O'就是这样造成的。需要在处理完-U参数后，让指针先往前指一位再处理就好了，不会再进行一次错误的参数处理了。top的这个Bug不是上游包引起的，而是Debian加这个补丁的时候引进去的。

为此我写了补丁并提交。