Re: [developer] review: 9599 Buffer overflow in usr/src/lib/libsocket/inet/ruserpass.c

2018-06-15 Thread Garrett D'Amore
I was going to make the same reply that JBK did — snprintf() error result
should be checked, to detect truncation.


On Fri, Jun 15, 2018 at 12:36 PM Toomas Soome 
wrote:

> hi!
> 
> please review.
> 
> issue: https://www.illumos.org/issues/9599
> rb: https://www.illumos.org/rb/r/1106/
> 
> thanks,
> toomas

--
openzfs: openzfs-developer
Permalink: 
https://openzfs.topicbox.com/groups/developer/T2dec2e813f846709-Mf744ff00df3a6af89911f688
Delivery options: https://openzfs.topicbox.com/groups


[developer] Re: [openzfs/openzfs] Trying again with sequential scan -- this is simply (#648)

2018-06-15 Thread Igor K
what is progress?
i tried to test it and have issues with some panic too with the same as with 
zfs tests.
ideas - how to try to fix it?


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/648#issuecomment-397590931
--
openzfs: openzfs-developer
Permalink: 
https://openzfs.topicbox.com/groups/developer/T6e124cbb2c8562a8-M45beb8d3694210ba1414cb52
Delivery options: https://openzfs.topicbox.com/groups


[developer] Re: [openzfs/openzfs] 9577 remove zfs_dbuf_evict_key tsd (#645)

2018-06-15 Thread Andrew Stormont
andy-js approved this pull request.

I've seen the deadlock.



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/645#pullrequestreview-129120299
--
openzfs: openzfs-developer
Permalink: 
https://openzfs.topicbox.com/groups/developer/Tb73a3d2cba6f4c23-M10c2c23c8a9d5ca7738b3092
Delivery options: https://openzfs.topicbox.com/groups