28 Oct
2008
28 Oct
'08
6:53 p.m.
KR> + tcp_info = fopen(path, "r"); KR> + if (tcp_info== NULL) { Whitespace before the == ... KR> + port_list.list = malloc(count * sizeof(struct vnc_port *)); Why not allocate the space for the structs too? KR> + port_list.list[port_list.cur] = malloc(sizeof(struct vnc_port)); If you do the above, then you won't have to allocate here. Otherwise I think it looks fine. -- Dan Smith IBM Linux Technology Center Open Hypervisor Team email: danms@us.ibm.com