blob: abc89761f3321d918149f50e8d1a032af70bfcb6 (
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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
|
[[!meta copyright="Copyright © 2010 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]]."]]"""]]
suckless-tools
==============
Home of dwm, dmenu and other quality software with a focus on simplicity, clarity, and frugality.
**Home page**: [[http://suckless.org]]
[[!toc startlevel=2]]
* * *
Log
---
* **Started**: 2012-01-31
* **Discussed**: [2012-01-31](http://lists.debian.org/debian-hurd/2012/01/msg00191.html)
* **Draft Submitted**: [2012-02-01](http://lists.debian.org/debian-hurd/2012/02/msg00001.html)
* **Submitted**: 2012-02-02, Bug#[658386](http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=658386)
* **Accepted**: -
* * *
ToDo
----
Here is the output of `grep -R PATH_MAX suckless-tools-38/*`:
dmenu/dmenu_path.c: char buf[PATH_MAX];
* * *
Comments
--------
Here are comments on the patch...
void
scan(void) {
- char buf[PATH_MAX];
+ char *buf = NULL;
+ char *new_buf = NULL;
+ size_t buf_size = 0;
In this function we turn `buf` into dynamically allocated string.
while((ent = readdir(dp))) {
+ buf_size = strlen(dir)+strlen(ent->d_name)+2;
+ if(!buf || strlen(buf) < buf_size) {
+ new_buf = realloc(buf, buf_size);
+ if(!new_buf)
+ die("realloc failed");
+ buf = new_buf;
+ }
For each directory entry we create or adapt the buffer size.
}
+ free(buf);
We free the buf when done.
|
