[systemd-devel] [PATCH] util: Fix signedness error in lines(), match implementations

Colin Walters walters at verbum.org
Thu Jan 1 12:35:26 PST 2015


On Thu, Jan 1, 2015, at 03:21 PM, Zbigniew Jędrzejewski-Szmek wrote:

> Is there an actual regression? Afaict, your patch does not change
> the behaviour in any way...

I can't think of an actual real world regression.  I could have said
"error introduced by" or something?  It only fixes the case where
the ioctl(fd, TIOCGWINSZ, &ws) fails for some reason - we would
end up thinking the screen had some very large number of lines.

(I only noticed this as I was copying the code)



More information about the systemd-devel mailing list