summaryrefslogtreecommitdiff
path: root/service_solahart_jakarta_selatan__082122541663/gnumach_integer_overflow.mdwn
blob: 08a292683e92f47866cbe6fd769988abdd691bf2 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
[[!meta copyright="Copyright © 2012, 2013 Free Software Foundation, Inc."]]

[[!meta license="""[[!toggle id="license" text="GFDL 1.2+"]][[!toggleable
id="license" text="Permission is granted to copy, distribute and/or modify this
document under the terms of the GNU Free Documentation License, Version 1.2 or
any later version published by the Free Software Foundation; with no Invariant
Sections, no Front-Cover Texts, and no Back-Cover Texts.  A copy of the license
is included in the section entitled [[GNU Free Documentation
License|/fdl]]."]]"""]]

[[!tag open_issue_gnumach]]


# IRC, freenode, #hurd, 2012-07-04

    <braunr> yes, we have integer overflows on resident_page_count, but
      luckily, the member is rarely used

See also [[gnumach_vm_object_resident_page_count]].


## IRC, freenode, #hurd, 2013-06-04

    <elmig> this is declared as int on vm_object.h
    <elmig> and as it as counter it's always positive
    <braunr> yes it should be unsigned
    <elmig> ok
    <braunr> but leave it as it is for consistency with the rest
    <elmig> i send patch :)
    <braunr> please no
    <braunr> unless you've fully determined the side effects
    <elmig> i've grepped the vars and saw only comparisons > and = 0
    <elmig> never less than 0
    <braunr> > 0 is the same
    <braunr> well
    <braunr> > not, but >= would be a problem
    <elmig> http://paste.debian.net/plain/8527
    <elmig> asctually no >=0
    <braunr> still, i don't want to change that unless it's strictly necessary
    <braunr> hum, you're grepping ref_count, not resident_page_count
    <elmig> i did both
    <elmig> on resident_page_count theres resident_page_count >= 0
    <elmig> = 0, == 0
    <braunr> this isn't the only possible issue
    <braunr> anyway
    <braunr> for now there is no reason to change anything unless you do a full
      review
    <elmig> only place i see resdent_page_count and page_count being decreased
      it's on vm/vm_resident.c
    <elmig> vm_page_remove() and vm_page_replace()