diff options
author | Brian Behlendorf <[email protected]> | 2016-10-21 08:23:27 -0700 |
---|---|---|
committer | GitHub <[email protected]> | 2016-10-21 08:23:27 -0700 |
commit | e4ffa98dcaf2208c742609f0ab2bdb343071446a (patch) | |
tree | 8b60321630f98db86dac20ad4e66a24372fcd409 /module/zcommon | |
parent | 9523b15ac1881946a7a6b7a145c8e2bb622dc56c (diff) |
Fix userquota_compare() function
The AVL tree compare function requires that either -1, 0, or 1 be
returned. However the strcmp() function only guarantees that a
negative, zero, or positive value is returned. Therefore, the
return value of strcmp() needs to be sanitized with AVL_ISIGN.
This was initially overlooked because the x86_64 implementation
of strcmp() happens to only returns the allowed values. This
was observed on an aarch64 platform which behaves correctly but
differently as described above.
Reviewed-by: Jinshan Xiong <[email protected]>
Reviewed-by: Richard Laager <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #5311
Closes #5313
Diffstat (limited to 'module/zcommon')
0 files changed, 0 insertions, 0 deletions